diff --git a/PLANET.md b/PLANET.md index 4b858cc9..7587ff8b 100644 --- a/PLANET.md +++ b/PLANET.md @@ -68,7 +68,7 @@ java -Xmx100g -Xms100g \ --download-threads=10 --download-chunk-size-mb=1000 \ `# Also download name translations from wikidata` \ --fetch-wikidata \ - `# Personal preference overrides to default OpenMapTiles schema` \ + `# Personal preference overrides from OpenMapTiles schema (these are default now)` \ --transportation-name-brunnel=false --transportation-z13-paths=true \ --mbtiles=output.mbtiles \ --nodemap-type=sparsearray --nodemap-storage=ram 2>&1 | tee logs.txt diff --git a/config-example.properties b/config-example.properties index 4528d5a8..d00d84f9 100644 --- a/config-example.properties +++ b/config-example.properties @@ -126,17 +126,17 @@ # boundary_country_names=false #### "transportation"/"transportation_name" layer -# Include paths and path names at z13 (default is only 14) -# transportation_z13_paths=true +# Set minimum zoom for paths and path names to z14 (default is 13) +# transportation_z13_paths=false # We attempt to merge any road name lines with the same attributes and touching endpoints, # but any 3-way nodes and intersections prevent line merging. # To prevent merging from combining onramps/offramps or opposite directions of divided highways, set: # transportation_name_limit_merge=true -# To exclude "brunnel" attribute from road name line features so that tunnels and bridges don't -# prevent merging: -# transportation_name_brunnel=false +# By default, the "brunnel" attribute is excluded from from road name line features so that tunnels +# and bridges don't prevent merging. To include "brunnel" tag: +# transportation_name_brunnel=true # To set a shorter length limit for road name lines so you can render a shield, but not the full name: # transportation_name_size_for_shield=true diff --git a/flatmap-basemap/README.md b/flatmap-basemap/README.md index 7eba8e78..e6936fcd 100644 --- a/flatmap-basemap/README.md +++ b/flatmap-basemap/README.md @@ -3,6 +3,15 @@ This basemap profile is based on [OpenMapTiles](https://github.com/openmaptiles/openmaptiles) v3.12.2. See [README.md](../README.md) in the parent directory for instructions on how to run. +## Differences from OpenMapTiles + +- `transportation_name` layer does not abbreviate road names yet +- `poi` layer missing the `agg_stop` tag +- Paths are visible at z13 in `transportation` and `transportation_name` layers instead of z14 in OpenMapTiles, to + revert this behavior set `--transportation-z13-paths=false` +- `brunnel` tag is excluded from `transportation_name` layer to avoid breaking apart long `transportation_name` + lines, to revert this behavior set `--transportation-name-brunnel=true` + ## Code Layout [Generate.java](./src/main/java/com/onthegomap/flatmap/basemap/Generate.java) generates code in diff --git a/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/Transportation.java b/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/Transportation.java index 2f8929f4..c9d016ef 100644 --- a/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/Transportation.java +++ b/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/Transportation.java @@ -123,7 +123,7 @@ public class Transportation implements boolean z13Paths = config.arguments().getBoolean( "transportation_z13_paths", "transportation(_name) layer: show paths on z13", - false + true ); MINZOOMS = Map.of( FieldValues.CLASS_TRACK, 14, diff --git a/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/TransportationName.java b/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/TransportationName.java index ad91c9d0..fb97c21c 100644 --- a/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/TransportationName.java +++ b/flatmap-basemap/src/main/java/com/onthegomap/flatmap/basemap/layers/TransportationName.java @@ -142,7 +142,7 @@ public class TransportationName implements this.brunnel = config.arguments().getBoolean( "transportation_name_brunnel", "transportation_name layer: set to false to omit brunnel and help merge long highways", - true + false ); this.sizeForShield = config.arguments().getBoolean( "transportation_name_size_for_shield", @@ -157,7 +157,7 @@ public class TransportationName implements boolean z13Paths = config.arguments().getBoolean( "transportation_z13_paths", "transportation(_name) layer: show paths on z13", - false + true ); MINZOOMS = Map.of( FieldValues.CLASS_TRACK, 14, diff --git a/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/BasemapTest.java b/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/BasemapTest.java index 0c756e41..eb6a9cff 100644 --- a/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/BasemapTest.java +++ b/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/BasemapTest.java @@ -200,7 +200,7 @@ public class BasemapTest { assertNumFeatures("transportation_name", Map.of( "name", "Boulevard du Larvotto", "class", "primary" - ), 14, 22, LineString.class); + ), 14, 12, LineString.class); } @Test diff --git a/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/layers/TransportationTest.java b/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/layers/TransportationTest.java index adda5da0..a8b5253f 100644 --- a/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/layers/TransportationTest.java +++ b/flatmap-basemap/src/test/java/com/onthegomap/flatmap/basemap/layers/TransportationTest.java @@ -37,7 +37,7 @@ public class TransportationTest extends AbstractLayerTest { "name", "", "_buffer", 4d, "_minpixelsize", 0d, - "_minzoom", 14, + "_minzoom", 13, "_maxzoom", 14 ), Map.of( "_layer", "transportation_name", @@ -48,7 +48,7 @@ public class TransportationTest extends AbstractLayerTest { "name_int", "Lagoon Path", "name:latin", "Lagoon Path", "_minpixelsize", 0d, - "_minzoom", 14, + "_minzoom", 13, "_maxzoom", 14 )), result); assertFeatures(13, List.of(Map.of( @@ -143,7 +143,7 @@ public class TransportationTest extends AbstractLayerTest { "ref", "90", "ref_length", 2, "network", "us-interstate", - "brunnel", "bridge", + "brunnel", "", "_minzoom", 6 )), features); @@ -166,7 +166,7 @@ public class TransportationTest extends AbstractLayerTest { "ref", "90", "ref_length", 2, "network", "us-interstate", - "brunnel", "bridge", + "brunnel", "", "_minzoom", 6 )), features); } @@ -184,7 +184,7 @@ public class TransportationTest extends AbstractLayerTest { "_layer", "transportation_name", "name", "North Washington Street", "class", "primary_construction", - "brunnel", "bridge", + "brunnel", "", "_minzoom", 12 )), process(lineFeature(Map.of( "highway", "construction", @@ -239,13 +239,13 @@ public class TransportationTest extends AbstractLayerTest { "mtb_scale", "4", "surface", "unpaved", "bicycle", "yes", - "_minzoom", 14 + "_minzoom", 13 ), Map.of( "_layer", "transportation_name", "class", "path", "subclass", "path", "name", "Path name", - "_minzoom", 14 + "_minzoom", 13 )), process(lineFeature(Map.of( "highway", "path", "mtb:scale", "4",