Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Batch fetching page texts #268

Open
RheingoldRiver opened this issue May 19, 2021 · 0 comments
Open

Batch fetching page texts #268

RheingoldRiver opened this issue May 19, 2021 · 0 comments

Comments

@RheingoldRiver
Copy link
Member

Hi, I implemented this in my wrapper library a couple days ago: RheingoldRiver/mwcleric@7d0ee4a There's no support for anything like this in mwclient right now, right? Is this something that could be added?

I'd ideally like to adapt this and add it to site.allpages maybe if you supply a batch_size parameter and generator is True, but it could also be a separate method similar to what I've done - but I think something like this belongs in mwclient itself rather than a wrapper (see also #55). (Though maybe I'm just missing where this is already supported and could have saved the time to write this if I asked first lol)

Anyway assuming it's not currently supported and you would like it added, I'm happy to adapt my implementation, how would you like it done? Or if you would like, you can feel free to adapt my code as well if you want!

(also could you merge #264 btw? I've been needing to use move_subpages a bunch, I think that would be useful to have in the library)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants