search

geopandas / contextily

Context geo-tiles in Python
https://contextily.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
493 stars 81 forks source link

expand retryer to include image reading #236

Closed martinfleis closed 4 months ago

martinfleis commented 4 months ago

moving image opening to the retryer in hope that it could resolve rare occasions of UnidentifiedImageError popping up. Reported in https://github.com/networkx/networkx/issues/7301 and seen once in our CI (here). I wasn't able to reproduce the issue locally in any form so I am not sure if it will fix it but it is the only thing I could think of.

To get a better control of retries, we should also probably expose wait, max_retries, n_connections and use_cache on add_basemap level, not only in bounds2img but I'll do that separately.

martinfleis commented 4 months ago

3770fda just updates the expected values changed in the test tiles. same issue is on main, just fixing this to get green CI. irrelevant to the contents here.