-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Replace BLAS with packed_simd #82
Comments
hora have used |
Hi @salamer! I was looking at hora when submitted this issue;) As a matter of fact, BLAS is a pain point when it comes to compilation. I also would like to avoid one unnecessary vector copy which is only used due to ndarray package. I would probably have to spend some more time on this topic and choose fast and at the same time easy to use package for SIMD. |
Different approach to consider https://github.com/zhao-lang/redis_hnsw/blob/master/src/hnsw/metrics.rs |
BLAS removed in https://github.com/qdrant/qdrant/releases/tag/v0.7.0 |
https://github.com/rust-lang/packed_simd
Is your feature request related to a problem? Please describe.
BLAS
is currently used for a single purpose - to make dot production faster. It looks like overkill.Compile time is pretty high and it complicates the development
Describe the solution you'd like
Replace
BLAS
withpacked_simd
library, which appear to be much lighter.Describe alternatives you've considered
Use some other lightweight library for SIMD (idk which one)
Additional context
None
The text was updated successfully, but these errors were encountered: