faiss icon indicating copy to clipboard operation
faiss copied to clipboard

Published 20 hours ago • verified publisher icon facebookresearch

install doc of c_api is out of date

Open Greastate opened this issue 3 years ago • 1 comments

Faiss version: dbc3d1d54bb17a7c4a67c6b6ba014dc10d2314ef

The install documentation of c_api is outdated, and the compilation cannot be successfully executed according to the steps inside. Now the compilation of c_api can be completed by adding compilation options(-DFAISS_ENABLE_C_API=ON), instead of entering the c_api directory and executing the make command. Please update the documentation, or delete c_api/install.md. , the confusion caused by the current documentation will waste a lot of time for users.

Thinks!

Greastate avatar Aug 23 '22 09:08 Greastate

True, the details in c_api/INSTALL.md were applicable before the project started using CMake, and it has not been updated since then. Would you like to send in a pull request with the changes that you propose?

Enet4 avatar Aug 23 '22 10:08 Enet4