mapbox / tile-cover

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

Invalid Tile When Longitude = 180 #75

Open lily-chai opened 7 years ago

lily-chai commented 7 years ago

Using tile-cover on Fiji at {min_zoom = 4, max_zoom = 4}, I got

[ [ 0, 8, 4 ], [ 15, 8, 4 ], [ 16, 8, 4 ] ]

[ 16, 8, 4 ] is not a valid z4 tile.

mourner commented 7 years ago

Also likely to be fixed with an upstream change to tilebelt — see https://github.com/mapbox/tilebelt/pull/27.

DenisCarriere commented 7 years ago

Another fix was submitted for this issue https://github.com/mapbox/tilebelt/pull/32

johnk1 commented 3 years ago

When integrating the upstream change from tilebelt, the tests will also need to be updated to support right-hand winding now produced by tilebelt (v1.0.2)