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

Render platforms ontop of pedestrian areas #4527

Closed
swedneck opened this issue Mar 27, 2022 · 10 comments
Closed

Render platforms ontop of pedestrian areas #4527

swedneck opened this issue Mar 27, 2022 · 10 comments

Comments

@swedneck
Copy link

swedneck commented Mar 27, 2022

Expected behavior

Platforms should be fully visible when part of a pedestrian area

Actual behavior

Platforms are obscured by pedestrian areas

@imagico
Copy link
Collaborator

imagico commented Mar 27, 2022

We render platforms as part of the road layers, as a result platforms mapped with linear ways are rendered above pedestrian areas but below pedestrian linear roads due to z-order:

pedestrian = {z = 310, roads = false},
raceway = {z = 300, roads = false},
motorway_link = {z = 240, roads = true},
trunk_link = {z = 230, roads = true},
primary_link = {z = 220, roads = true},
secondary_link = {z = 210, roads = true},
tertiary_link = {z = 200, roads = false},
service = {z = 150, roads = false},
track = {z = 110, roads = false},
path = {z = 100, roads = false},
footway = {z = 100, roads = false},
bridleway = {z = 100, roads = false},
cycleway = {z = 100, roads = false},
steps = {z = 90, roads = false},
platform = {z = 90, roads = false}

We have had many discussions if and how to change that (see for example: #527, #3281, #3872, #4221) - but with no definitive conclusion so far.

The semantics of mapping at the location you showed are of course fairly questionable - this is not the kind of mapping we would want to model our rendering after. That however does not mean that the problem of inconsistent drawing order between polygons and linear ways visible here is not an issue that would be good to address.

Side note: Trying to keep secret the location of your example is both pointless and counterproductive of course:

link removed

I suggest we close this because it is only a specific concrete manifestation of a problem we have existing issues open about.

@swedneck
Copy link
Author

swedneck commented Mar 29, 2022

well thanks for doxxing me, that's very appreciated. You are aware this violates your own CoC, right?

@amandasaurus
Copy link

amandasaurus commented Mar 29, 2022

@imagico 🙄 c'mon dude. If someone is trying to preserve their privacy, even if it's only a fig leaf, it's a bit mean to blatently rip that fig leaf away for no reason.

(edit 2022-05-31 to remove accidental typo “not”)

@matkoniecz
Copy link
Contributor

@imagico can you edit out that location comment? Yes, it is trivially recoverable, but pointing this out can be done without making even more accessible what someone has tried to not reveal (even if they failed).

Though doxxing is usually reserved for stronger actions especially as in this case it was really trivial to recover.

@imagico
Copy link
Collaborator

imagico commented May 18, 2022

@imagico can you edit out that location comment?

Sure.

Assessing an issue like this without determining the location is not possible of course. So as a general advice to anyone who wants to report an issue that they can only find at a specific location in the map and who at the same time does not want this location to be known to anyone else: The best way to do so - if you cannot find the same problem at any other location - is to try reproducing it in a test database and then posting the test data. If you post a screenshot from the actual map without a link to the location there are only two possibilities:

  • the developers here determine the location (which takes time not available for development work).
  • the problem can't be addressed because no one other than the person reporting the issue can analyze what causes it.

@matkoniecz
Copy link
Contributor

Also, note that in general if map view includes even a single name / ref code then it is almost always easy to find where it was located.

Even "bicycle parking near greengrocer and butcher" would allow locating given place.

If worried about privacy: find equivalent example in some remote location.

@mmd-osm
Copy link

mmd-osm commented May 18, 2022

It's not exactly trivial to find an equivalent example, so here's a first try:

(1) https://www.openstreetmap.org/#map=19/48.40266/10.00024 --> https://www.openstreetmap.org/way/248783286
(2) https://www.openstreetmap.org/#map=19/52.42961/7.06583
issue_4527

I used the following Overpass query for this stuff. I hope that's similar enough to the location described by the OP.

Assumption here is that there's a non-closed highway=pedestrian way, which is so close to a highway=platform, that parts of the platform are no longer being rendered.

[bbox:{{bbox}}];

way[highway=pedestrian][area=yes];

way(area)[highway=platform][ref](if:!is_closed())->.a;
foreach .a -> .b {

way(around.b:3)[highway=pedestrian](if:!is_closed());
if (count(ways) > 0) {
  out geom;
  .b out geom;
};
}

@mmd-osm
Copy link

mmd-osm commented May 31, 2022

As a reminder to everyone here, if you want to make sure that previous versions of your comments are really no longer accessible, you need to follow this description: https://docs.github.com/en/communities/moderating-comments-and-conversations/tracking-changes-in-a-comment

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jun 3, 2022

As a reminder to everyone here, if you want to make sure that previous versions of your comments are really no longer accessible, you need to follow this description: https://docs.github.com/en/communities/moderating-comments-and-conversations/tracking-changes-in-a-comment

Unfortunately, even in that case your original comments will still be visible to users who follow the respository with email notifications. For example, I still have the original 3 comments to this issue in my email inbox, including the image and the link which have been subsequently removed. It is important to only share information on Github which you are willing to have publically available to anyone on the internet.

Also, note that editing a comment does not notify people who follow by email. For example, the statement above: "You are aware this violates your own CoC, right?” was not visible to me or other maintainers who follow issues vis email, since that phrase was not originally included in the comment and was only added later.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jun 3, 2022

This issue is similar to #4221 and more broadly #3281, and cannot be easily solved without causing other problems, due to complexities in how road/pedestrian areas are rendered in relation to linear roads and platforms. I believe we should close this issue as a duplicate of those 2 earlier issues.

@pnorman pnorman closed this as not planned Won't fix, can't repro, duplicate, stale Jun 4, 2022
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

No branches or pull requests

7 participants