search

mapbox / tile-cover

Generate the minimum number of tiles to cover a geojson geometry
MIT License
189 stars 40 forks source link

[failing] test edgeline #34

Closed yhahn closed 9 years ago

yhahn commented 9 years ago

geometry-collection

This test fails because the intersection between tile and geometry ends up as a GeometryCollection upstream in turf-intersect and this check prevents any legit tiles from continuing on with a GeometryCollection:

A refactor of this to handle GeometryCollection is pretty ugly, and a better fix for this may be the work in scanline which appears to handle this case more robustly.

morganherlocker commented 9 years ago

Good catch. I logged an issue upstream in turf-intersect, where this will not be difficult to refactor.

https://github.com/Turfjs/turf-intersect/issues/3

Not a huge deal for now, as scanline will make this moot in tile-cover.