Skip to content

Commit

Permalink
Merge pull request #1116 from tilezen/zerebubuth/add-overlaps-bbox-fi…
Browse files Browse the repository at this point in the history
…lter

Zerebubuth/add overlaps bbox filter
  • Loading branch information
zerebubuth authored Nov 2, 2016
2 parents 84583cf + 9cac608 commit cd216db
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 22 deletions.
18 changes: 0 additions & 18 deletions integration-test/291-483-suppress-historical-closed.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,21 +15,3 @@
assert_has_feature(
17, 20931, 50616, 'pois',
{'id': 241643507, 'kind': 'closed', 'min_zoom': 17})

## US Naval Hospital (historical)
# https://www.openstreetmap.org/relation/317369

# original polygon should be present at z15
assert_has_feature(
15, 5266, 12666, 'landuse',
{'id': -317369, 'kind': 'hospital'})

# but POI should not be present
assert_no_matching_feature(
15, 5266, 12666, 'pois',
{'id': -317369})

# but it should be present at z17
assert_has_feature(
17, 21063, 50665, 'pois',
{'id': -317369, 'kind': 'historical'})
6 changes: 3 additions & 3 deletions integration-test/857-move_barriers_to_landuse.py
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
# update landuse to include barriers features and delete from boundaries

# city_wall in landuse
# http://www.openstreetmap.org/way/258909996
# http://www.openstreetmap.org/way/81522922
assert_has_feature(
12, 3302, 1750, 'landuse',
12, 2030, 1300, 'landuse',
{ 'kind': 'city_wall'})

# city_wall not in boundaries
assert_no_matching_feature(
12, 3302, 1750, 'boundaries',
12, 2030, 1300, 'boundaries',
{ 'kind': 'city_wall'})

# citywalls in landuse
Expand Down
2 changes: 1 addition & 1 deletion queries/boundaries.jinja2
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ SELECT
FROM planet_osm_polygon

WHERE
{{ bounds['line']|bbox_filter('way',3857) }} AND
{{ bounds['line']|bbox_overlaps('way',3857) }} AND
{% if zoom >= 16 %}
mz_boundary_min_zoom IS NOT NULL
{% else %}
Expand Down

0 comments on commit cd216db

Please sign in to comment.