geopandas / xyzservices

Source of XYZ tiles providers
https://xyzservices.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
150 stars 29 forks source link

IGN (cx.providers.GeoportailFrance.orthos) migrated their services (wxs.ign.fr to data.geopf.fr) #165

Closed salahelfarissi closed 3 months ago

salahelfarissi commented 3 months ago

IGN just changed their routing this month (March 2024). https://geoservices.ign.fr/services-geoplateforme-diffusion

try:
    cx.add_basemap(
        m.ax,
        source=cx.providers.GeoportailFrance.orthos,
        attribution=True,
        crs=Maps.CRS.GOOGLE_MERCATOR,
        zoom=zoom_level,
    )
except HTTPError as e:
    logger.error("Failed to load basemap: %s", e)
    logger.warning("Continuing without basemap.")
martinfleis commented 3 months ago

Thanks for the report! It seems that our automatic tooling actually picked it up in https://github.com/geopandas/xyzservices/commit/f31e55732dd4cba65453af9a5ffa21e75a5dac63. I will cut a release of xyzservices tomorrow to get it out.

martinfleis commented 3 months ago

I was actually wrong, we the tooling needs to be migrated.

@HaudinFlorence would you be able to look at it? You did the original implementation.

salahelfarissi commented 3 months ago

I can work on it if it suits you. I already work on a web platform that generates reports and we use their services.

image image

martinfleis commented 3 months ago

@salahelfarissi That would be very welcome. The code fetching the data currently lives here https://github.com/geopandas/xyzservices/blob/e8198dac0c10435c617dff3720aea3ec43561de4/provider_sources/_compress_providers.py#L87-L211

HaudinFlorence commented 3 months ago

@martinfleis I can have a look if needed. @salahelfarissi Don't hesitate to tell me if you need some help

salahelfarissi commented 3 months ago

@HaudinFlorence @martinfleis My first try. Please review and let me know if I need to write proper tests.

166

martinfleis commented 3 months ago

@salahelfarissi it should be covered by existing test suite