updates for min_size

planetiler-custommap/README.md

@@ -222,6 +222,8 @@ A feature is a defined set of objects that meet a specified filter criteria.
 - `exclude_when` - A [Boolean Expression](#boolean-expression) which determines if a feature that matched the include
   expression should be skipped. If unspecified, no exclusion filter is applied.
 - `min_zoom` - An [Expression](#expression) that returns the minimum zoom to render this feature at.
+- `min_size` - An [Expression](#expression) that returns the minimum length of line features or square root of the
+  minimum area of polygon features to emit below the maximum zoom-level of the map.
 - `attributes` - An array of [Feature Attribute](#feature-attribute) objects that specify the attributes to be included
   on this output feature.
 
@@ -672,7 +674,8 @@ docker run -v "$(pwd)/data":/data ghcr.io/onthegomap/planetiler:latest verify /d
   - `geometry` - Geometry type of the expected output feature.
   - `min_zoom` - Min zoom level that the output feature appears in.
   - `max_zoom` - Max zoom level that the output feature appears in.
-  - `min_size` - Minimum length of line features or square root of the minimum area of polygon features to emit below the maximum zoom-level of the map.
+  - `min_size` - Minimum length of line features or square root of the minimum area of polygon features to emit below
+    the maximum zoom-level of the map.
   - `tags` - Attributes expected on the output vector tile feature, or `null` if the attribute should not be set. Use
     `allow_extra_tags: true` to fail if any other tags appear besides the ones specified here.
   - `allow_extra_tags` - If `true`, then fail when extra attributes besides tags appear on the output feature.

planetiler-custommap/planetiler.schema.json

@@ -392,6 +392,10 @@
           "items": {
             "$ref": "#/$defs/attribute"
           }
+        },
+        "min_size": {
+          "description": "Minimum length of line features or square root of the minimum area of polygon features to emit below the maximum zoom-level of the map",
+          "$ref": "#/$defs/expression"
         }
       }
     },

planetiler-custommap/planetilerspec.schema.json

@@ -85,7 +85,7 @@
         },
         "min_size": {
           "description": "Minimum length of line features or square root of the minimum area of polygon features to emit below the maximum zoom-level of the map",
-          "type": "double"
+          "type": "number"
         },
         "tags": {
           "description": "Attributes expected on the output vector tile feature, or null if the attribute should not be set. Use allow_extra_tags: true to fail if any other tags appear besides the ones specified here",

planetiler-custommap/src/main/java/com/onthegomap/planetiler/custommap/validator/SchemaValidator.java

@@ -208,6 +208,7 @@ public class SchemaValidator {
           validate(prefix + ".layer", issues, expected.layer(), actual.getLayer());
           validate(prefix + ".minzoom", issues, expected.minZoom(), actual.getMinZoom());
           validate(prefix + ".maxzoom", issues, expected.maxZoom(), actual.getMaxZoom());
+          validate(prefix + ".minsize", issues, expected.minSize(), actual.getMinPixelSizeAtZoom(expected.atZoom()));
           validate(prefix + ".geometry", issues, expected.geometry(), GeometryType.typeOf(actual.getGeometry()));
           Set<String> tags = new TreeSet<>(actualTags.keySet());
           expected.tags().forEach((tag, value) -> {

planetiler-custommap/src/main/resources/samples/shortbread.spec.yml

@@ -221,6 +221,7 @@ examples:
   - layer: streets
     geometry: line
     min_zoom: 8
+    min_size: 0
     tags:
       bridge: false
       kind: primary
@@ -262,6 +263,7 @@ examples:
     geometry: line
     min_zoom: 8
     allow_extra_tags: false
+    min_size: 0
     tags:
       bridge: false
       kind: primary
@@ -291,6 +293,7 @@ examples:
     geometry: line
     min_zoom: 8
     allow_extra_tags: false
+    min_size: 0
     tags:
       bridge: false
       kind: rail
@@ -707,6 +710,7 @@ examples:
     layer: boundaries
     geometry: line
     min_zoom: 0
+    min_size: 0
     tags:
       maritime: true
       admin_level: 2
@@ -723,6 +727,7 @@ examples:
     layer: boundaries
     geometry: line
     min_zoom: 7
+    min_size: 0
     tags:
       maritime: false
       admin_level: 4

planetiler-custommap/src/main/resources/samples/shortbread.yml

@@ -132,7 +132,7 @@ layers:
   - source: osm
     geometry: line
     # TODO get min admin level from relations
-    min_size: 0.0
+    min_size: 0
     min_zoom:
       default_value: 7
       overrides:
@@ -380,7 +380,7 @@ layers:
   features:
   - source: osm
     geometry: line
-    min_size: 0.0
+    min_size: 0
     min_zoom:
       default_value: 13
       overrides:

planetiler-custommap/src/test/java/com/onthegomap/planetiler/custommap/ConfiguredFeatureTest.java

@@ -1022,4 +1022,29 @@ class ConfiguredFeatureTest {
     )));
     assertEquals("example.com_file.osm.pbf", loadConfig(config).sources().get(0).defaultFileUrl());
   }
+
+  @ParameterizedTest
+  @CsvSource("""
+    10,10
+    ${10+1},11
+    ${feature.tags.key}|9
+    """)
+  void setMinSize(String input, double output) {
+    var config = """
+      sources:
+        osm:
+          type: osm
+          url: geofabrik:rhode-island
+          local_path: data/rhode-island.osm.pbf
+      layers:
+      - id: testLayer
+        features:
+        - source: osm
+          min_size: %s
+          geometry: line
+      """.formatted(input);
+    testLinestring(config, Map.of("key", 9), feature -> {
+      assertEquals(output, feature.getMinPixelSizeAtZoom(11));
+    }, 1);
+  }
 }

planetiler-custommap/src/test/java/com/onthegomap/planetiler/custommap/validator/SchemaValidatorTest.java

@@ -77,6 +77,7 @@ class SchemaValidatorTest {
       features:
       - source: osm
         geometry: polygon
+        min_size: 10
         include_when:
           natural: water
         attributes:
@@ -121,6 +122,9 @@ class SchemaValidatorTest {
     "true,water,polygon,natural: water,allow_extra_tags: false",
     "true,water,polygon,,allow_extra_tags: true",
     "false,water,polygon,,allow_extra_tags: false",
+
+    "true,water,polygon,,min_size: 10",
+    "false,water,polygon,,min_size: 9",
   })
   void testValidateWaterPolygon(boolean shouldBeOk, String layer, String geometry, String tags, String allowExtraTags)
     throws IOException {