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

Colleges mapped as points appear very early #1181

Closed
wmisener opened this issue Dec 19, 2024 · 1 comment
Closed

Colleges mapped as points appear very early #1181

wmisener opened this issue Dec 19, 2024 · 1 comment
Labels
bug Something isn't working openmaptiles A change is needed in OpenMapTiles to support this

Comments

@wmisener
Copy link
Collaborator

wmisener commented Dec 19, 2024

Any college mapped as a point appears starting at z10, essentially the city level. However, colleges and universities mapped as areas appear based on its size, somewhere between z10 and z14 but typically later than z10. Since most major colleges are mapped as areas, this leads to the weird result that tiny colleges appear many zoom levels before major universities. Example:
Screenshot 2024-12-18 at 8 44 13 PM
UCLA Extension and other minor institutions appear at z10, while major universities like UCLA and USC don't appear till z13 and z14 respectively. Appears to be due to when the respective features appear in the upstream OMT tiles. I don't think this behavior was present when #1022 was merged, so maybe there was a more recent change to when point amenities appear in the tiles...

@wmisener wmisener added bug Something isn't working openmaptiles A change is needed in OpenMapTiles to support this labels Dec 19, 2024
@wmisener
Copy link
Collaborator Author

This was a bug in onthegomap/planetiler#1118 that excluded points when feature areas were calculated, fixed promptly by onthegomap/planetiler#1132 after being raised on Slack. Now the point colleges do not appear until z14, the previous (and I think desirable) behavior.
Screenshot 2024-12-20 at 8 22 12 AM
Thanks to Mike Barry for the quick fix on the planetiler side!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working openmaptiles A change is needed in OpenMapTiles to support this
Projects
None yet
Development

No branches or pull requests

1 participant