Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

buildings layer thining on z14 to z15 #2052

Merged
merged 22 commits into from
Feb 12, 2022
Merged
Changes from 9 commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
60 changes: 56 additions & 4 deletions queries.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,7 @@ layers:
- vectordatasource.transform.remove_feature_id
- vectordatasource.transform.truncate_min_zoom_to_1dp
sort: vectordatasource.sort.buildings
area-inclusion-threshold: 1
area-inclusion-threshold: 3
nvkelso marked this conversation as resolved.
Show resolved Hide resolved
pois:
geometry_types: [Point, MultiPoint, Polygon, MultiPolygon]
transform:
Expand Down Expand Up @@ -1714,7 +1714,7 @@ post_process:
params:
end_zoom: 15
nvkelso marked this conversation as resolved.
Show resolved Hide resolved
source_layers: [buildings]
pixel_area: 0.25
pixel_area: 1.0
nvkelso marked this conversation as resolved.
Show resolved Hide resolved

- fn: vectordatasource.transform.quantize_height
params:
Expand All @@ -1740,6 +1740,58 @@ post_process:
source_layer: buildings
target_value_type: int

- fn: vectordatasource.transform.drop_properties
params:
source_layer: buildings
start_zoom: 0
end_zoom: 17
nvkelso marked this conversation as resolved.
Show resolved Hide resolved
properties:
- landuse_kind
where: >-
landuse_kind not in ['aerodrome', 'airfield', 'cemetery', 'college', 'hospital', 'park', 'school', 'university']
nvkelso marked this conversation as resolved.
Show resolved Hide resolved

- fn: vectordatasource.transform.drop_properties
params:
source_layer: buildings
start_zoom: 0
end_zoom: 17
nvkelso marked this conversation as resolved.
Show resolved Hide resolved
properties:
- kind_detail
where: >-
kind_detail not in ['airport', 'civic', 'church', 'college', 'commercial', 'fire_station', 'industrial', 'military', 'hospital', 'office', 'public', 'prison', 'retail', 'school', 'sports_centre', 'stadium', 'station', 'synagogue', 'train_station', 'university', 'warehouse']
nvkelso marked this conversation as resolved.
Show resolved Hide resolved

- fn: vectordatasource.transform.drop_properties
params:
source_layer: buildings
start_zoom: 0
end_zoom: 16
properties:
- building_material
- layer
nvkelso marked this conversation as resolved.
Show resolved Hide resolved
- location
- osm_relation
- roof_color
- roof_height
- roof_material
- roof_orientation
- roof_shape
- root_id
- volume

- fn: vectordatasource.transform.drop_properties
params:
source_layer: buildings
start_zoom: 0
end_zoom: 17
nvkelso marked this conversation as resolved.
Show resolved Hide resolved
# short-hand for "name" property in the list below means all name-like
# properties.
all_name_variants: true
properties:
- name
- old_name
- addr_housenumber
- addr_street

# fit min zoom onto the scale rank, as we'll be using the scale rank rather
# than the min zoom to calculate whether a building should make it into a
# tile. having them split separately means we can't merge them together.
Expand Down Expand Up @@ -1779,9 +1831,9 @@ post_process:
# process may also have created small inners.
- fn: vectordatasource.transform.drop_small_inners
params:
end_zoom: 15
end_zoom: 16
nvkelso marked this conversation as resolved.
Show resolved Hide resolved
source_layers: [buildings]
pixel_area: 0.25
pixel_area: 1.0
nvkelso marked this conversation as resolved.
Show resolved Hide resolved

- fn: vectordatasource.transform.numeric_min_filter
params:
Expand Down