-
Notifications
You must be signed in to change notification settings - Fork 80
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
Retryer #35
Retryer #35
Conversation
Hmm, strange; works locally, but I removed I think this may have something to do with using |
I've finally had a look at the functionality, this is great. If you add (passing) tests, I'll merge this. Hopefully it can make it to the upcoming |
Not sure why this closed, but I had to reopen this to get the merge commit to show. Also, not sure how to mock this functionality in a unittest. |
OK fair enough. I guess it is tested it at least works in any other method that pulls down tiles. One more thing, should we then add |
I don't think requests is required by geopy, but it's used in a large amount of web-touching code. I also use it in cenpy. I mainly like it for the I've added it to the requirements. |
Cool! |
This adds functionality to wait/retry between web requests.
This is necessary for some map APIs, especially when downloading large/high-detail tilesets.
This makes contextily kind of like a general purpose tile downloader for any API in the tile-url format, and I've found it extremely handy to work with rate-limited scientific APIs.