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

Ann build not paralleled #3

Open
jianshu93 opened this issue Jan 8, 2022 · 2 comments
Open

Ann build not paralleled #3

jianshu93 opened this issue Jan 8, 2022 · 2 comments

Comments

@jianshu93
Copy link

jianshu93 commented Jan 8, 2022

Hello,

Thanks for this reimplementation. I noticed when running Ann.building() in the demo it is not paralleled. Apparently, the k-means tree can be paralleled right. any iter() could be change to par_iter(). For example, Lin 147 in ann.rs can be changed to .par_iter() using rayon package?

Thanks,

Jianshu

@uzushino
Copy link
Owner

Hi @jianshu93

any iter() could be change to par_iter(). For example, Lin 147 in ann.rs can be changed to .par_iter() using rayon package?

That's a good idea.
I don't know how to use random number generator in rayon::par_iter() but I'll give it some thought.

@jianshu93
Copy link
Author

Hello uzushino,

Any update on this parallel idea? I am thinking annoy in rust should be faster than the Spotify version with rayon par_iter().

Thanks,

Jianshu

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