Does not build

[ritchie46]

Does not build

[ritchie46]

Hi.. Could you tell me which example you are trying to run? I notice that the blas dependencies can be troublesome.

[ijsnow]

Sorry, coulda been more specific. I get the error when running make build in examples/neural-network

[ritchie46]

I believe the examples are not up to date anymore indeed. The lsh-rs library still builds on my side (I hope on yours as well). However the examples don't work anymore because my blas libraries don't compile. At the moment I am too busy too work that out. But as the main library still builds I believe it should work with updating some dependencies / toggling off blas. If you believe there are any issues with the main library let me know, I'll figure it out.

[bwindsor22]

Also seeing this error when I add lsh-rs = "0.4.0" to cargo.toml and run the install.

[bwindsor22]

Hey, so I managed to compile by pulling the code out of lsh-rs/lsh-rs/src/ and copying into my src, with some module changes.. you can see this commit here if it's helpful bwindsor22/thistle@477cd93

As an aside, for vector similarity lookup, you might also check out granne, hnswlib-rs, and hnsw libraries

[bwindsor22]

@ritchie46 thanks for the comment on blas, was helpful -- thoughts on shipping the examples separately? Something like Cargo.toml -> RemoveMe.Cargo.toml for a quick fix

[bwindsor22]

@ritchie46 err, looking back, some of the errors that @ijsnow saw were things I fixed locally -- e.g. changing this to std::marker::PhantomData

8 | use serde::export::PhantomData;
| ^^^^^^ private module

[bwindsor22]

... however when git clone and cargo make locally, I do get the blas error. So I think the blas error is an issue, but as I can get the code to compile without solving it, it might be possible to ship a working version of this lib only by quarantining the blas bit and separately solving the serde errors ijsnow was seeing

[ritchie46]

@ritchie46 err, looking back, some of the errors that @ijsnow saw were things I fixed locally -- e.g. changing this to std::marker::PhantomData

8 | use serde::export::PhantomData;
| ^^^^^^ private module

Yes, that's definitly a good one. Could you make a PR for that fix?

[ritchie46]

uarantining the blas bit and

This make sense, BLAS is only for squeezing out max performance, but should definitely be opt in.

[bwindsor22]

Great, submitted.