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

Enhancement: radius #39

Closed
tchaton opened this issue Dec 14, 2019 · 4 comments
Closed

Enhancement: radius #39

tchaton opened this issue Dec 14, 2019 · 4 comments
Labels

Comments

@tchaton
Copy link

tchaton commented Dec 14, 2019

Hey @rusty1s,

Check this repo: https://github.com/humanpose1/torch_radius_neighbors/tree/master
Nanoflann is faster / robust than scipy on cpu and remove the need to convert back from numpy to torch. Integrating the code might be interesting here :)
Also, do you have any idea on how to improve the radius search which is currently built on brute force search (it won't scale to large pointcloud)

Best,
Thomas Chaton

@rusty1s
Copy link
Owner

rusty1s commented Dec 14, 2019

Thank you for the pointer. As an alternative GPU computation, we should benchmark a KeOps implementation against the current approach

@tchaton
Copy link
Author

tchaton commented Dec 14, 2019

This paper seems pretty interesting for the task

semi_covex_hull_tree_icdm_accepted_final2.pdf

@tchaton
Copy link
Author

tchaton commented Dec 21, 2019

Hey @rusty1s,

This could be used to make radius search faster on gpu.
http://on-demand.gputechconf.com/gtc/2014/presentations/S4117-fast-fixed-radius-nearest-neighbor-gpu.pdf

@github-actions
Copy link

This issue had no activity for 6 months. It will be closed in 2 weeks unless there is some new activity. Is this issue already resolved?

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

No branches or pull requests

2 participants