Remove fork SQ in favor of vanilla FAISS version#1594
Conversation
3425441 to
71d071e
Compare
|
@alexanderguzhva 🔍 Important: PR Classification Needed! For efficient project management and a seamless review process, it's essential to classify your PR correctly. Here's how:
For any PR outside the kind/improvement category, ensure you link to the associated issue using the format: “issue: #”. Thanks for your efforts and contribution to the community!. |
71d071e to
3e4abb0
Compare
|
issue: #1501 |
3e4abb0 to
76c6c4b
Compare
Signed-off-by: Alexandr Guzhva <alexanderguzhva@gmail.com>
76c6c4b to
0f2f511
Compare
|
@alexanderguzhva e2e jenkins job failed, comment |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alexanderguzhva, foxspy The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
this should be supported by baseline PR facebookresearch/faiss#5128, which introduces RVV SQ in the vanilla faiss
Removes fork
ScalarQuantizerfacilities and uses the ones from the baseline