Test polygon decoding and encoding options

mapbox / mapnik-vector-tile

Mapnik implemention of Mapbox Vector Tile specification

BSD 3-Clause "New" or "Revised" License

553 stars 117 forks source link

Test polygon decoding and encoding options #165

Closed springmeyer closed 8 years ago

springmeyer commented 8 years ago

In geometry_visual_test we need to add testing of both the booleans:

multi_polygon_union
process_all_mp_rings

springmeyer commented 8 years ago

@flippmoke we should also settle on and document the reasons for defaults. FWIW I've seen no bad side effects yet of multi_polygon_union:true default. Ideally we can get some testcases in place with overlapping polygons so that we can capture whether this helps: e.g. can we demonstrate that unioning improves vtile size? Can we demonstrate that it helps or at least does not hurt the effort in #166?