elixir-nx / hnswlib

Elixir binding for the hnswlib library.
Apache License 2.0
52 stars 9 forks source link

Update hnswlib to v0.8.0 #17

Closed cocoa-xu closed 7 months ago

cocoa-xu commented 7 months ago

This PR updated hnswlib to v0.8.0, and added the following functions:

cocoa-xu commented 7 months ago

Restarted Windows CI as it seems stuck for more than half an hour... Not sure about the reason...

josevalim commented 7 months ago

Yeah, Windows is still getting stuck for some reason. :(

cocoa-xu commented 7 months ago

Yeah, Windows is still getting stuck for some reason. :(

Yeah... I'll cancel the workflow and try it with latest version of Erlang and Elixir. If it still stuck, I'll try it on a physical machine or later.

cocoa-xu commented 7 months ago

Finally I have access to a physical Windows machine. It seems that the problem only occurs when testing HNSWLib.BFIndex.delete_vector/2, i.e., mix test test/hnswlib_bfindex_test.exs:315

C:\Users\cocoa\Documents\GitHub\hnswlib>mix test test/hnswlib_index_test.exs
................................................
Finished in 2.3 seconds (0.00s async, 2.3s sync)
48 tests, 0 failures

Randomized with seed 921679

C:\Users\cocoa\Documents\GitHub\hnswlib>mix test test/hnswlib_bfindex_test.exs:315
# crashed

hnswlib_bfindex