search

gravitystorm / openstreetmap-carto

A general-purpose OpenStreetMap mapnik style, in CartoCSS
Other
1.51k stars 812 forks source link

Render tram level crossings #4296

Open ahmetlii opened 3 years ago

ahmetlii commented 3 years ago

See https://wiki.openstreetmap.org/wiki/Tag:railway%3Dtram_level_crossing and https://wiki.openstreetmap.org/wiki/Tag:railway%3Dtram_crossing . It could be good to render.

jeisenbe commented 3 years ago

These tags, railway=tram_crossing and railway=tram_level_crossing, are nowsupported by iD, CyclOSM and OpenTopoMap. Usage is increasing steadily since mid-2020, to about 10k uses for each tag.

TagHistory chart:

Screen Shot 2021-01-28 at 21 24 16

Taginfo comparison and maps:

Screen Shot 2021-01-28 at 21 23 17
hungerburg commented 3 years ago

Please think twice before rendering this tag. The on the ground feature that it maps is of no concern to most users. In the regions highlighted on the taginfo map above, it is very common to cross tram rails embedded into streets ten times a ride, even drive on them. The rendering must not look in any way like a rail crossing, just enough so mappers see, its mapped, if at all.

PS: I bet, the sudden explosion in usage is testament to the influence that the iD editor (its validator) has on openstreetmap, https://github.com/openstreetmap/iD/issues/5990#issuecomment-674419342

matkoniecz commented 2 years ago

The main problem described in https://github.com/openstreetmap/iD/issues/5990 is that iD asks to add them at positions where no actual crossing exists.

Not sure is it a reason to ignore a bad data and consider this tag as tainted and useless - or consider highlighting invalid data as useful part of a feedback to mappers.