Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Results consistently differ from the C++ library #21

Open
ag-tcm opened this issue Feb 10, 2021 · 3 comments
Open

Results consistently differ from the C++ library #21

ag-tcm opened this issue Feb 10, 2021 · 3 comments

Comments

@ag-tcm
Copy link

ag-tcm commented Feb 10, 2021

I did some testing comparing the results from this library and the C++ library and they are consistently different. I needed to get it to work so I went through line by line in a debugger comparing the two and fixed the java code on my local copy. Let me know if you'd like me to push it here so others can use it. Oh, also I added support for the Manhattan distance.

@Dawien
Copy link

Dawien commented Jul 15, 2021

I did some testing comparing the results from this library and the C++ library and they are consistently different. I needed to get it to work so I went through line by line in a debugger comparing the two and fixed the java code on my local copy. Let me know if you'd like me to push it here so others can use it. Oh, also I added support for the Manhattan distance.

I also encountered the same problem when I was using it. Can you open source the modified code for reference? Thank you very much!

@ag-tcm
Copy link
Author

ag-tcm commented Jul 15, 2021

No problem, I put my modified code here: https://github.com/ag-tcm/annoy-java.

@Dawien
Copy link

Dawien commented Jul 16, 2021

No problem, I put my modified code here: https://github.com/ag-tcm/annoy-java.

Great!! Thanks for sharing!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants