node-s2 icon indicating copy to clipboard operation
node-s2 copied to clipboard

Max_cell: 1 sometimes lead to bad results

Open mick opened this issue 11 years ago • 1 comments

Adding a failing test case here

Given the LLRect screen shot 2014-09-11 at 4 29 25 pm

and the options {min:1, max:30, max_cells:1}

I get back 2 cells, both are much larger then the need to be to cover the area. level 0 and 4.

In s2map the same options result in 1 cell at level 8 screen shot 2014-09-11 at 4 28 16 pm

cc: @tmcw

mick avatar Sep 11 '14 23:09 mick

It is for sure related to using S2LatLngRect, If I use a polygon, which is the only thing that s2map uses. I get the cover i expect. Add a test case that works with polygons

mick avatar Sep 13 '14 02:09 mick