shortbread-tiles / shortbread-docs

Documentation of the Shortbread vector tile schema
https://shortbread-tiles.org
Creative Commons Zero v1.0 Universal
25 stars 7 forks source link

Min zooms for railways don't make sense #47

Open pnorman opened 5 months ago

pnorman commented 5 months ago

The description to railway=rail in the streets layer says

ways with service=* on zoom level 8+, other ways on zoom level 10+

For railways service is yard, spur, siding, or crossover. These are less important than ways without a service tag, and styles like osm-carto render them as less significant. To include them on z8 before the main lines seems like an obvious error that the 8 and 10 got flipped.

cc @Nakaner

Nakaner commented 5 months ago

The documentation is wrong. The Tilemaker implementation writes all railway tracks on zoom level 10+ but railway=rail/narrow_gauge without service=* on zoom level 8+.

Writing unimportant features on lower zoom levels than important features would not make sense.