5 Commits
Author SHA1 Message Date
Matthijs Douze 8d1b964c11 Enable dynamic dispatch build on Windows (#5127)
Summary:
Pull Request resolved: https://github.com/facebookresearch/faiss/pull/5127

Attempt to enable FAISS dynamic dispatch (DD) mode on Windows/MSVC.

Changes:
- CMakeLists.txt: Remove if(NOT WIN32) guard from DD section, add MSVC per-file
  SIMD flags (/arch:AVX2, /arch:AVX512, /bigobj) alongside existing GCC/Clang flags
- build-pull-request.yml: Add windows-x86_64-DD-cmake job that runs immediately
  (no dependency on linux-x86_64-cmake), builds with MSVC and FAISS_OPT_LEVEL=dd,
  runs C++ tests and Python tests

This diff is expected to fail on Windows due to MSVC requiring explicit template
specialization declarations (C++ §17.8.3) which GCC/Clang don't enforce. The CI
failure will surface the exact errors to guide the fix.

UPDATE: in fact, MSVC does not seem to require this. After fixing build scripts and a few classical Windows error, the C++ and python tests pass.

Reviewed By: algoriddle

Differential Revision: D101649751

fbshipit-source-id: 765ef8483d02652ce58625cd22baab8870acf718
2026-04-23 13:31:11 -07:00
Michael Norris eff0898a13 Enable linting: lint config changes plus arc lint command (#3966)
Summary:
Pull Request resolved: https://github.com/facebookresearch/faiss/pull/3966

This actually enables the linting.

Manual changes:
- tools/arcanist/lint/fbsource-licenselint-config.toml
- tools/arcanist/lint/fbsource-lint-engine.toml

Automated changes:
`arc lint --apply-patches --take LICENSELINT --paths-cmd 'hg files faiss'`

Reviewed By: asadoughi

Differential Revision: D64484165

fbshipit-source-id: 4f2f6e953c94ef6ebfea8a5ae035ccfbea65ed04
2024-10-22 09:46:48 -07:00
Gergely Szilvasy c577f43be6 fix perf test (#3255)
Summary: Pull Request resolved: https://github.com/facebookresearch/faiss/pull/3255

Reviewed By: mlomeli1

Differential Revision: D53811550

fbshipit-source-id: b103dc2d0cf43cabb9e4e7e74f4e737ff39fbbbc
2024-02-15 10:38:55 -08:00
Gergely Szilvasy 873b1bc68c fix perf test (#3253)
Summary:
Pull Request resolved: https://github.com/facebookresearch/faiss/pull/3253

relaxing thresholds + allow equal, duh

Reviewed By: mlomeli1

Differential Revision: D53803000

fbshipit-source-id: 7b68386523d9a44a0cd8e099fc1d72b3ed7d5e43
2024-02-15 04:08:09 -08:00
Gergely Szilvasy f2620119f0 fix omp parallelism in fast scan range search
Summary: Fix omp n^2 parallelism

Reviewed By: mdouze

Differential Revision: D53705601

fbshipit-source-id: 3fcc2368c436185119f6e988ee2867dfd7d8eb07
2024-02-13 12:21:00 -08:00