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

Stamen terrain-background max zoom #368

Closed
vpipkt opened this issue Sep 23, 2020 · 4 comments
Closed

Stamen terrain-background max zoom #368

vpipkt opened this issue Sep 23, 2020 · 4 comments

Comments

@vpipkt
Copy link

vpipkt commented Sep 23, 2020

It appears that the max zoom is 11 for the Stamen terrain-background variant. Unsure how to verify this.

https://github.com/leaflet-extras/leaflet-providers/blob/master/leaflet-providers.js#L348

vpipkt added a commit to vpipkt/leaflet-providers that referenced this issue Sep 23, 2020
Closes leaflet-extras#368 

Zoom levels here are based on observing 404 response for a very small sample of locations, at various zoom levels.
@brunob
Copy link
Member

brunob commented Sep 23, 2020

i don't think so... http://maps.stamen.com/terrain/#17/48.38355/-4.49551

@vpipkt
Copy link
Author

vpipkt commented Sep 23, 2020

@brunob I am specifically highlighting the terrain-background variant. With that specific variant I get some 404s and some 200s at zoom levels less than the max zoom given. Is there a reference from stamen about the max zoom ?

curl -i  http://tile.stamen.com/terrain-background/14/4499/6312.png

@brunob
Copy link
Member

brunob commented Sep 24, 2020

I can get these tiles too zoom 15 here http://maps.stamen.com/terrain-background/#15/48.3835/-4.4955

FTR, terrain background was introduced by #50 & terrain worldwide by #223 and here is a stamen file reporting zoom range http://maps.stamen.com/js/tile.stamen.js

I think the bug come from stamen tile server cache which may be a bit broken. Maybe you can contact theme to confirm this ?

@vpipkt
Copy link
Author

vpipkt commented Sep 24, 2020

@brunob thank you for the research and background info! Closing this.

@vpipkt vpipkt closed this as completed Sep 24, 2020
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 a pull request may close this issue.

2 participants