Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

Max_cell: 1 sometimes lead to bad results #77

Open
mick opened this issue Sep 11, 2014 · 1 comment
Open

Max_cell: 1 sometimes lead to bad results #77

mick opened this issue Sep 11, 2014 · 1 comment
Labels

Comments

@mick
Copy link
Contributor

mick commented Sep 11, 2014

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 mick added the bug label Sep 11, 2014
@mick
Copy link
Contributor Author

mick commented Sep 13, 2014

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant