-
Notifications
You must be signed in to change notification settings - Fork 6
Implement S2CellIndex and S2ClosestCellQuery #2
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
Implement S2CellIndex and S2ClosestCellQuery #2
Conversation
|
Am a little nervous here, public tests for S2 are not great. Current coverage stats: Collected with |
|
Fantastic thanks @benjamin-cooper-ai it might take me some time to review, thanks for all your work |
|
Hey thanks so much @benjamin-cooper-ai, I had a quick look over it and it's a very welcome addition to the library 🎉 Docs available here: https://s2js.org/classes/s2js.s2.CellIndex.html |
|
🎉 This PR is included in version 1.44.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This is an implementation of S2CellIndex. Making this useful required implementing S2ClosestCellQuery and S2ClosestEdgeQuery.
I tried to test this as well as possible - I ported over a union of tests from the C++ library, the Golang library, and added some other example tests that match what those libraries return.
Let me know what I can add to build confidence here!
Fixes #1