Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cellunion contains method not working #23

Closed
vivek8690 opened this issue Jun 24, 2020 · 1 comment
Closed

cellunion contains method not working #23

vivek8690 opened this issue Jun 24, 2020 · 1 comment

Comments

@vivek8690
Copy link

for [ -122.435577, 37.751363] this location I am creating radius covering[s2.RegionCoverer.getRadiusCovering] cellunion with 500 m radius
and I am trying that cell union contains this point [-122.434339,37.751433 ]
It returns false every time
I tried same scenario for other locations as well but it was working for them

@jkao
Copy link
Collaborator

jkao commented Jan 28, 2021

Hi Vivek thanks for submitting an issue!

This is dependent on the parameters you pass to RegionCoverer as that will determine the S2 Covering.

You can manually test the region covering here: https://s2.sidewalklabs.com/regioncoverer/

For me to debug, could you post a code snippet of your experiment?

@jkao jkao closed this as completed Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants