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

Align WebMercatorQuad TMS with mercantile and GDAL #80

Closed
vincentsarago opened this issue Feb 16, 2022 · 0 comments · Fixed by #81
Closed

Align WebMercatorQuad TMS with mercantile and GDAL #80

vincentsarago opened this issue Feb 16, 2022 · 0 comments · Fixed by #81
Labels
enhancement New feature or request question Further information is requested

Comments

@vincentsarago
Copy link
Member

as mentioned in #77 and #63 the WebMercatorQuad TMS which is aligned with the OGC doc is not as precise as the one used in GDAL or mercantile (it all comes down to the definition of the earth half size

# GDAL
https://github.com/OSGeo/gdal/blob/35c07b18316b4b6d238f6d60b82c31e25662ad27/gcore/tilematrixset.cpp#L79
>> 20037508.342789244

# Morecantile 
https://github.com/developmentseed/morecantile/blob/master/morecantile/data/WebMercatorQuad.json#L9
>> 20037508.3427892

Pro

  • align with mercantile
  • alight with GoogleMapsCompatible TMS

Cons

  • COG created with rio-cogeo's Web Optimized will have different origins (but with the change, they should be aligned with GDAL ones)
  • TiTiler tiles might be slightly different

cc @geospatial-jeff

@vincentsarago vincentsarago added enhancement New feature or request question Further information is requested labels Feb 16, 2022
@vincentsarago vincentsarago changed the title [breaking] Align WebMercatorQuad TMS with mercantile and GDAL Align WebMercatorQuad TMS with mercantile and GDAL Feb 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant