-
Notifications
You must be signed in to change notification settings - Fork 823
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
render icon and icon+text for large POIs on lower zoom levels #1057
Comments
I have been experimenting with this, but it gives some unexpected results.
I think we should keep rendering all POI on z17, because z18 is too late for small rural shops. What would you use as threshold? I think 3000 pixels (the threshold for labels) is too high for this to have any effect, because hardly any POI reaches 3000 pixels before z17. |
Also note that 67 % van of the supermarkets and 62 % of the department stores are tagged as node only. |
I think the basic problem is that buildings with multiple POIs within are generally mapped as nodes inside of the building way, and these will have zero area. I could see adding rendering at lower zooms to large features, but I don't think we want to take away at higher zooms. This will also tend to render more car shops, as they often have an outside lot which is part of the shop (e.g. http://www.openstreetmap.org/way/158991197) while most shops are just inside. You'll also see more shops of types which tend to be large freestanding buildings, e.g. supermarkets and theaters. This is probably okay, as they tend to be more significant. |
Not sure if I agree - I don't think taking away small museums from z16 would be bad, for example. |
See http://www.openstreetmap.org/?mlat=41.9025&mlon=12.4534#map=15/41.9025/12.4534 http://www.openstreetmap.org/?mlat=49.6216&mlon=6.1412#map=16/49.6216/6.1412 http://www.openstreetmap.org/?mlat=51.5146&mlon=-0.1530#map=15/51.5146/-0.1530
http://www.openstreetmap.org/?mlat=51.5805&mlon=4.7282#map=15/51.5805/4.7282
see also #1055
superset of #1022 #1020 #1019
The text was updated successfully, but these errors were encountered: