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

Map tiles fail to load at max zoom #32

Closed
ThorntonMatthewD opened this issue Oct 17, 2023 · 6 comments
Closed

Map tiles fail to load at max zoom #32

ThorntonMatthewD opened this issue Oct 17, 2023 · 6 comments
Labels
good first issue Good for newcomers

Comments

@ThorntonMatthewD
Copy link
Collaborator

The maximum zoom on the map is currently set to 20, however at this zoom level the map tiles appear as solid grey images.

Screenshot_2023-10-16_20-00-39

You can see it in action here: https://hackgvl.github.io/open-map-data-multi-layers-demo/?lat=34.851224882536776&lng=-82.39967912435533&zoom=20&maps=free-wi-fi-hotspots

I'm wondering if the max zoom needs to be 18 instead since it seems to be inclusive.

@oliviasculley
Copy link
Member

There's actually also a bigger issue with tiles in that we should probably find a better alternative than freeloading off of open street map's donation-based tile servers, which could change their policy at any time: https://operations.osmfoundation.org/policies/tiles/

@ThorntonMatthewD
Copy link
Collaborator Author

My vote is for Spinal Map for the remainder of October. 🎃🤘

That is a really good point... I've seen some projects store their base tiles in a GitHub repo, but I'm not sure what limits we'd run into there in the long run.

@Alex-Grimes
Copy link
Member

Spinal Map looks amazing 😆. I could do some research on alternative options for tile servers. Is a PR to lower the zoom limit to 18 still wanted?

@ThorntonMatthewD
Copy link
Collaborator Author

I think for now that would be good. If/when a different source of tiles is put into place then perhaps it can be increased again (if supported).

@Alex-Grimes Alex-Grimes mentioned this issue Oct 27, 2023
@ThorntonMatthewD
Copy link
Collaborator Author

#36 fixes #32

Thank you, @Alex-Grimes !!!

@allella
Copy link
Member

allella commented Jan 19, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Development

No branches or pull requests

4 participants