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

Consider nblast-rs #1

Open
clbarnes opened this issue Jan 16, 2024 · 2 comments
Open

Consider nblast-rs #1

clbarnes opened this issue Jan 16, 2024 · 2 comments

Comments

@clbarnes
Copy link

Or if there's anything nblast-rs is missing, raise an issue! https://crates.io/crates/nblast

@clbarnes clbarnes changed the title Rebase nblast on nblast-rs Consider nblast-rs Jan 16, 2024
@schlegelp
Copy link
Owner

I mainly added a simple NBLAST implementation to this library to (a) try out bosque for NN calculations and (b) benchmark performance against that in NBLAST in navis - turns out gains are minimal but I like how easy multithreading is in rust.

I also freely admit that I didn't understand half the rust code in nblast-rs 😅

If fastcore-rs ever makes it from the "Philipp tinkers with Rust" stage to something actually useful, I'll definitely look into using nblast-rs.

@clbarnes
Copy link
Author

clbarnes commented Jan 16, 2024

Learning rust by tinkering with connectomics libraries nobody will ever use is a perfectly valid strategy ;)

Bosque does look interesting - that developer's 3rd ANN library! Working on slices makes it trickier to use with nblast-rs but it's probably worth a benchmark.

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