gravitystorm / openstreetmap-carto

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

Render `railway=yard` #4844

Open marioxcc opened 1 year ago

marioxcc commented 1 year ago

railway=station is rendered even when it is a freight station (railway:traffic_mode=freight). It makes sense to render yards too. Freight trains collect freight from industries (i.e.: stations) into a nearby yards, then haul a long string of wagons between yards, then distribute it to nearby industries. Thus, yards are as important or even more than the freight stations.

dch0ph commented 10 months ago

For railway=station, it is fairly obvious that it is associated with a specific location (more or less at station buildings), whereas railway=yard seems to more of a landuse, despite the wiki describing it as a node-only tagging.

A name attached to an area landuse=railway will already be rendered. Frankly, this would seem to be a better way of rendering something that is by definition an area.

There seems to be little harm in duplicating the name between the railway=yard node and an enclosing landuse=railway?