mapbox / tile-cover

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

extra tiles #57

Closed morganherlocker closed 9 years ago

morganherlocker commented 9 years ago

Extra tiles are possible, if fairly rare, in the current implementation. They occur in cases with high/low slope segments that span many tiles. In practical cases, I do not think we will run into many cases where this will matter. I think that the imprecision is introduced in getTileSegments.

Labeling as wontfix, but we can revisit if necessary.

cc @yhahn @mourner