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

add GoogleMapsCompatible TMS to match GDAL TMS #77

Closed
wants to merge 1 commit into from

Conversation

vincentsarago
Copy link
Member

ref developmentseed/titiler#431 (comment)

There are small differences between GDAL GoogleMapsCompatible and the WebMercatorQuad TMS defined by morecantile. It all comes down to the value of the earth HALF_CIRCUMFERENCE defined as

# 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

I'm not sure I want to modify the WebMercatorQuad so IMO, adding a new TMS seems to be the better solution.

Note: we need a better name than GoogleMapsCompatible

@vincentsarago
Copy link
Member Author

This is in fact not needed for rio-tiler nor titiler because rio-tiler is smart enough to handle small rounding errors

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

Successfully merging this pull request may close these issues.

1 participant