sqlite-vss icon indicating copy to clipboard operation
sqlite-vss copied to clipboard

[BUG] Missing `faiss::FaissException` in the compile procedure

Open yfyang86 opened this issue 2 years ago • 1 comments

[BUG] Missing faiss::FaissException in the compile procedure

  • FAISS Version: Current master as 2023-08-16 faiss
  • sqlite-vss Version: Current master

Behavior:

Error message

no type named 'FaissException' in namespace 'faiss'

Fix, add the <faiss/impl/FaissException.h> file to sqlite-vss.cpp:

...
#include <random>

#include <faiss/impl/FaissException.h>
#include <faiss/IndexFlat.h>
#include <faiss/IndexIVFPQ.h>
#include <faiss/impl/AuxIndexStructures.h>
...

yfyang86 avatar Aug 16 '23 05:08 yfyang86

If you checkout the submodules in the sqlite-vss repo, it points to the facebookresearch/faiss@868e17f29493075742170885f1f57c7b9e61d9ea , which is pretty old. We can bump this to something newer (at least v1.7.4, Apr 19th ), where we can update the #includes to fix errors like this.

But if you're compiling sqlite-vss yourself, I recommend checking out the submodules in that repo, since we haven't tested on newer version of Faiss.

asg017 avatar Aug 16 '23 17:08 asg017