Matthijs Douze
Matthijs Douze
Related to this SWIG issue https://github.com/swig/swig/issues/2709
Hmmm that's weird, we don't observe that.
I prefer trying to repro this on docker + linux only. It is surprising that the CI does not catch this since it since it also uses docker and linux....
@mnorris11 can you try to repro on a AWS machine?
OK the current version is complex but less repetitive. I'll import to fbcode. Weird that the errors still appear. I thought I relaxed these tests. Did you rebase on latest...
Do you have numbers (even very rough) that show where this modification speeds up search?
What is the current status of this PR?
Let's consider this diff is a draft for now.
Thanks for the report, could you give a reproduction example ?
Do you have a reference implementation of the Jaccard metric to compare with?