Tile size: water only changes

integration-test/1730-further-water-layer-name-dropping.py

@@ -211,13 +211,11 @@ def test_label_lake_athabasca_z5(self):
             }),
         )
 
-        # we should get a label placement point, and its zoom should have been
-        # adjusted. we should also have all the names at this point.
+        # we should also have all the names at this point.
         self.assert_has_feature(
             z, x, y, 'water', {
                 'kind': 'lake',
-                'label_placement': True,
-                'min_zoom': 5,
+                'min_zoom': 2,  # min_zoom changed at https://github.com/tilezen/vector-datasource/pull/2010/
                 'name': str,
                 'name:de': str,
             })

integration-test/1838-too-many-bays.py

@@ -33,9 +33,9 @@ def _bay(osm_id, area, name):
         )
 
         with self.features_in_tile_layer(z, x, y, 'water') as features:
-            # should only have top 10 features
-            self.assertEqual(10, len(features))
+            # should only have top 2 features
+            self.assertEqual(2, len(features))
 
             # check that they're in order
             ranks = [f['properties']['kind_tile_rank'] for f in features]
-            self.assertEqual(range(1, 11), ranks)
+            self.assertEqual(range(1, 3), ranks)

queries.yaml

@@ -794,6 +794,29 @@ post_process:
       where: >-
         kind == 'lake'
 
+  # remove names per kind for a given zoom
+  # before generating any label placements
+  - fn: vectordatasource.transform.drop_properties
+    params:
+      source_layer: water
+      start_zoom: 0
+      end_zoom: 16
+      # short-hand for "name" property in the list below means all name-like
+      # properties.
+      all_name_variants: true
+      properties:
+        - name
+      where: >-
+        (kind == 'sea' and zoom < 4) or
+        (kind == 'lake' and zoom < 5) or
+        (kind == 'river' and zoom <  12) or
+        (kind == 'canal' and zoom <  13) or
+        (kind == 'dam' and zoom <  14) or
+        (kind == 'stream' and zoom <  14) or
+        (kind == 'ditch' and zoom <  15) or
+        (kind == 'drain' and zoom <  15) or
+        (properties is not None and properties.get('is_tunnel') is True)
+
   - fn: vectordatasource.transform.handle_label_placement
     params:
       layers:
@@ -822,22 +845,58 @@ post_process:
       # table.
       min_zoom: >-
         [min_zoom for min_zoom, area_threshold in [
-            (4,  40000000000),
-            (5,  10000000000),
-            (6,   5000000000),
-            (7,    400000000),
-            (8,    200000000),
-            (9,    100000000),
-            (10,    10000000),
-            (11,     4000000),
-            (12,      750000),
-            (13,      100000),
-            (14,       50000),
-            (15,       10000),
-            (16,        None),
+            (1, 1000000000000),
+            (2,  500000000000),
+            (3,  250000000000),
+            (4,  120000000000),
+            (5,   80000000000),
+            (6,   40000000000),
+            (7,   10000000000),
+            (8,     500000000),
+            (9,     200000000),
+            (10,     40000000),
+            (11,      8000000),
+            (12,      1000000),
+            (13,       500000),
+            (14,        50000),
+            (15,        10000),
+            (16,         None),
         ] if area >= area_threshold][0]
       where: >-
-        kind == 'lake' and label_placement
+        kind in ['lake','bay','water','riverbank','reservoir'] and label_placement
+
+  # now that we have the label points
+  # drop most water properties at lower zooms
+  # this is for lines and polygons
+  - fn: vectordatasource.transform.drop_properties
+    params:
+      source_layer: water
+      start_zoom: 8
+      end_zoom: 15
+      properties:
+        - id
+        - area
+        - layer
+        - wikidata_id
+        - kind_tile_rank
+        - label_placement
+      where: >-
+        geom_type in ('LineString', 'MultiLineString', 'Polygon', 'MultiPolygon')
+
+  # now that we have the label points
+  # drop water names at lower zooms for polys only
+  - fn: vectordatasource.transform.drop_properties
+    params:
+      source_layer: water
+      start_zoom: 8
+      end_zoom: 15
+      # short-hand for "name" property in the list below means all name-like
+      # properties.
+      all_name_variants: true
+      properties:
+        - name
+      where: >-
+        geom_type in ('Polygon', 'MultiPolygon')
 
   - fn: vectordatasource.transform.handle_label_placement
     params:

yaml/water.yaml

@@ -25,9 +25,15 @@ globals:
       key: { col: way_area }
       op: '>='
       table:
-        # OSM data starts at z8, but we should include some stuff at z7 so that
+        # OSM data starts at z8, but we should include some stuff at earlier so
         # the transition doesn't have a gap in it.
-        - [ 7,   160000000 ]
+        - [ 1, 20000000000 ]
+        - [ 2, 10000000000 ]
+        - [ 3,  5000000000 ]
+        - [ 4,  2000000000 ]
+        - [ 5,   500000000 ]
+        - [ 6,   200000000 ]
+        - [ 7,    50000000 ]
         - [ 8,    20000000 ]
         - [ 9,     5000000 ]
         - [ 10,    1000000 ]