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

Allow filters on quadtree.find #19

Open
bmschmidt opened this issue Jun 13, 2017 · 1 comment · May be fixed by #28
Open

Allow filters on quadtree.find #19

bmschmidt opened this issue Jun 13, 2017 · 1 comment · May be fixed by #28
Assignees

Comments

@bmschmidt
Copy link

I like to use an optional filter argument with the find method so that the search can return only the nearest relevant point (e.g.: the nearest red point if blue points have been temporarily hidden by a user interaction). The basic fork that works for my purposes is extremely simple; it would take slightly more work to properly handle coincident points.

Something like this is referenced in an existing pull discussion. Happy to file a pull request for just this if desired.

@mbostock mbostock self-assigned this Jun 13, 2017
Fil added a commit that referenced this issue Jun 5, 2020
Fil added a commit that referenced this issue Jun 5, 2020
@Fil Fil linked a pull request Jun 5, 2020 that will close this issue
@Fil
Copy link
Member

Fil commented Jun 5, 2020

Implemented by #28 ; please test & comment!

demo @ https://observablehq.com/d/07d5853f05318fb2

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

Successfully merging a pull request may close this issue.

3 participants