-
Notifications
You must be signed in to change notification settings - Fork 819
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
Rendering barriers above highway areas #2394
Conversation
We intend to merge highway areas into linear highway fill at some point - how will this interact? |
I don't know. You mean the lua branch? |
No, #2046 and the related incomplete PR |
Once #2058 is more or less in its releasable shape, I'm happy to test it. But since it looks like you don't have a time to resume this work, I think we can merge this one (if there are no other problems) and look again when the time will come. |
I am a bit concerned that this PR would mean that hedges etc will cover (and perhaps sometimes completely hide) roads. That might be worse than the bug this is trying to solve? |
1 similar comment
I am a bit concerned that this PR would mean that hedges etc will cover (and perhaps sometimes completely hide) roads. That might be worse than the bug this is trying to solve? |
I will check it. |
We already show hedges along the roads and I see no such problem. It does not hurt even pedestrian and bicycle ways (which are thinner than roads), for example: https://www.openstreetmap.org/way/370601787 Let me know if that's what you were asking me, because it seems it has nothing to do with highway areas at all. If it's OK for you, I will fix the code to make it ready to be merged. |
Taken from here. |
Now I see... It's hard for me to judge, both problems look pretty important for me. It would be good to have at least the estimation how often they would appear, but I don't know how to measure it. |
I think this is undesirable and should be solved by splitting up barriers in things that could show up over roads and ones that shouldn't. Hedges are things that shouldn't show up over roads. Walls could. |
If there will be no objection, I will try to do the split. |
I'm not sure whether walls (and citywalls even more so) wouldn't cause the same problem. |
Should we find some examples to test it or we just want to avoid it anyway? |
Testing in real-life examples never hurts but I'm quite sceptical. |
So I'm closing it now. The change is rather simple, so we could easily recreate it once we need it. |
I can see here just one example of barrier hiding the road, but I can give you a lot more of examples, when barrier is not visible, and it could make map user wrong:
I think the situation of barrier hiding the road is a lot lot rarer, than situation when lack of barrier line could make someone walk/ ride in wrong direction |
I think it haven't been tested enough (just one example of barrier hiding the road!) and should be reopen. |
Due to technical limitations on GitHub it can't be reopened, but the change is simple to do - see #2394 (comment). Is there anyone willing to test it? |
Resolves #528.
The code is simple - I've just moved 2 barrier layers (line-barriers and area-barriers) after 3 filling layers (highway-area-fill, roads-fill and turning-circle-fill) and it works, but I don't know if there are side effects, so please report any problems.
Warsaw, z19:
Before
After