[Tile-serving] [openstreetmap/osm2pgsql] Don't pre-filter relation member tags (#721)

Sarah Hoffmann notifications at github.com
Sat Mar 18 20:14:35 UTC 2017


Tags for ways of multipolygons are no longer prefiltered before handing them into the relation member filter function. For new-style MPs the tags are not relevant anyways. For old-style MPs, the filter now first creates a list of common tags and then filters the result with the standard relation tags filter.

I have adapted style.lua but noticed that old-style MPs are handled slightly differently. The tags of the MP are computed as the union of all tags on the out ways while the C transform computes them as the intersection of tags. I haven't touched multi.lua yet.

Tests pass but it could possibly do with visual verification that results are the same.    

Fixes #605 and #659.

You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/osm2pgsql/pull/721

-- Commit Summary --

  * don't pre-filter relation member tags
  * filter member tags when rechecking for old style members
  * adapt lua style to unfiltered relation members

-- File Changes --

    M geometry-processor.cpp (13)
    M geometry-processor.hpp (2)
    M osmtypes.hpp (2)
    M output-multi.cpp (19)
    M output-pgsql.cpp (21)
    M style.lua (16)
    M tagtransform-c.cpp (206)
    M tagtransform-c.hpp (19)
    M tagtransform-lua.cpp (21)
    M tagtransform-lua.hpp (15)
    M tagtransform.hpp (20)

-- Patch Links --

https://github.com/openstreetmap/osm2pgsql/pull/721.patch
https://github.com/openstreetmap/osm2pgsql/pull/721.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/721
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20170318/5c409d95/attachment.html>


More information about the Tile-serving mailing list