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

Layering regression: highway=pedestrian area now covers building=roof #885

Closed
kkofler opened this issue Aug 16, 2014 · 3 comments
Closed

Comments

@kkofler
Copy link

kkofler commented Aug 16, 2014

At:
https://www.openstreetmap.org/?mlat=48.21949&mlon=16.46611#map=19/48.21949/16.46611
(Vienna, Austria, subway U2 Donauspital), we have an area with highway=pedestrian, area=yes and no layer tag, and a building=roof with layer=3 (3 rather than 1 because it matches the subway rails which go above things with layer=2). The roof used to be rendered above the pedestrian area, which matched reality. Now the pedestrian area covers the roof in the rendering. A building=station with no layer tag (that actually rests on the pedestrian area in reality) also gets covered by the pedestrian area in the rendering.

@daganzdaanda
Copy link

I would turn the building=station and the highway=pedestrian into a multipolygon. That will make the station show up again, and it's the desired tagging.
You do have a point that building=roof with a layer greater than the pedestrian area should be put on top. That's a bug, see also #688 and #140.
So we could close this as a duplicate and continue at #688?

@kkofler
Copy link
Author

kkofler commented Aug 19, 2014

Yes, it's clearly the same, so we can close this one as a duplicate.

@kkofler kkofler closed this as completed Aug 19, 2014
@kkofler
Copy link
Author

kkofler commented Aug 19, 2014

For what it's worth, this particular example should be fixed/worked around by: https://www.openstreetmap.org/changeset/24861806

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

No branches or pull requests

3 participants