-
Notifications
You must be signed in to change notification settings - Fork 821
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
Do not render railway=level_crossing for tramways #4559
Comments
Use of railway=crossing/railway=level_crossing on trams seems to be common practice world wide. So there is definitely no consensus not to do so.
It would be - we do that for turning circles at the moment (rendering them in a design adjusted to the intersecting road). Considering how widely these are mapped and how significant they are often for navigation i would be skeptical about removing rendering. But adjusting the starting zoom level or the symbol specifically for tram crossings might be a good idea. |
Beware, that this tagging very likely is not something, that individuals knowingly applied. For some time, the validator of the iD editor did add the tag to intersections tram/highway, when people answered "Yes" to the question "Do you want to connect the ways?" Meanwhile, the validator uses https://wiki.openstreetmap.org/wiki/Tag:railway%3Dtram_level_crossing instead. in the history graph you can easily spot, when the change happened. That makes me wonder, how /organic/ the other numbers are. http://taghistory.raifer.tech/#***/railway/tram_level_crossing&***/railway/level_crossing shows some coincidence: As tram rises, the uptake of railway slows down. I'd say, this can be fixed in tagging. |
Let's have a broader view on the topic. It's the very nature of a map to show ways of any kind. When ways cross, the crossings are depicted by - surprise - ways that cross each other. Streets cross streets, bicycle lanes cross footpaths, bike paths cross streets, etc. In none of these cases, we show a special marker for the crossing, (not even for marked crosswalks so far). In my opinion, a street crossing a tramway track is no different (at least in the usual case, where there are no special signs, barriers, or traffic lights). Of course, a level crossing of an actual railway with a road is a different story. It usually has physical features associated with it, such as barriers and/or traffic lights. For navigation, it is very important as it may increase travel time significantly. So the tagging and the rendering is absolutely valid for actual railway. The same holds for tram level crossing when an additional tag like crossing:barrier/bell/light/saltire is set.
I also think that most of the respective tagging occurred due to respective suggestions / warnings by the editors, which used to be too unspecific. In the meantime, the editors seem to have switched to tram_level_crossing, which we do not render. So one solution would be to replace level_crossing by tram_level_crossing everywhere. But it does not feel okay. tram_level_crossing is a disputed tag, and whether the level crossing is for trams can be determined automatically. So I would love to solve the problem by having the renderer making that decision automatically. |
I have opened a PR that would improve the rendering as suggested. Rendering tram crossings in case there are traffic lights mapped, would require an extra DB column, though, AFAICT. |
Expected behavior
railway=level_crossing should not be rendered if it is part of a way tagged railway=tram
Actual behavior
railway=level_crossing is always rendered as a large diagonal cross. For tramways, which naturally have many level crossing, this pollutes the map, and in certain areas dominates the look of the map, without adding much useful information. This is particularly bad for printed maps, like the ones generated by MapOSMatic.
Links and screenshots illustrating the problem
https://www.openstreetmap.org/#map=16/47.5462/7.5634&layers=N
It was debated whether to map level crossings for tramways, with a tendency to not do so:
https://lists.openstreetmap.org/pipermail/tagging/2017-April/031941.html
I'm not sure this is implementable with the software infrastructure for carto. However, e.g. in Basel, there are hundred of them mapped already, so it would be awkward to remove them manually.
The text was updated successfully, but these errors were encountered: