-
Notifications
You must be signed in to change notification settings - Fork 3.9k
[DO NOT MERGE] Bisect CMake version that resulted in regression #3925
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
KyleFromNVIDIA
wants to merge
259
commits into
facebookresearch:main
Choose a base branch
from
KyleFromNVIDIA:cmake-bisect
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 163 commits
Commits
Show all changes
259 commits
Select commit
Hold shift + click to select a range
2c9e965
style check
divyegala 2e434fe
add required methods
divyegala 382c178
conditionally compile cagra
divyegala 8675974
copyTo and copyFrom
divyegala c7fcf4a
style check
divyegala eae832d
Merge branch 'main' into raft-cagra-hnsw
divyegala 4b76e5f
Merge branch 'main' into raft-cagra-hnsw
divyegala 065f912
add read/write
divyegala 301f429
Merge remote-tracking branch 'origin/raft-cagra-hnsw' into raft-cagra…
divyegala 2b0ea76
add destructor
divyegala 8c83bd2
destructor body, copyto reset
divyegala 39fb35a
remove destructor
divyegala 49e2610
move cmake sources around
divyegala 11bf6b2
merge upstream
divyegala d4434bb
more protections for copying
divyegala ac65c2d
support default constructed IndexHnswCagra in copyTo
divyegala 619c376
fix failing binary hnsw tests
divyegala e25f8a4
link faiss_gpu target to OpenMP
divyegala e835150
raft still can't find openmp
divyegala aeabe12
openmp flags and uint32 IndexType
divyegala 4e80586
forgot conditional check in index_read
divyegala c4bcaba
minor changes
divyegala 341a3fc
api change
divyegala 0ae7702
Merge branch 'raft-api-changes' into raft-cagra-hnsw
divyegala 172aa65
working python
divyegala 0cd684e
compile option to swig
divyegala 7ff8b3b
expose ivf pq params
divyegala 66d236f
update comments style
divyegala f697eac
Merge remote-tracking branch 'upstream/main' into raft-cagra-hnsw
divyegala 1d6e6b1
use raft::runtime where possible
divyegala 4a01ad4
format
divyegala 949e634
format properly
divyegala bccd54a
InnerProduct
divyegala 320654c
Merge remote-tracking branch 'upstream/main' into raft-cagra-hnsw
divyegala 2aaa6e9
passing ip tests
divyegala 70b0ab8
address review
divyegala e5756cc
Merge remote-tracking branch 'upstream/main' into raft-cagra-hnsw
divyegala 8f72ce4
Merge remote-tracking branch 'upstream/main' into raft-cagra-hnsw
divyegala 4148fea
base level only search
divyegala c13bcff
Merge remote-tracking branch 'upstream/main' into raft-cagra-hnsw
divyegala 24a555d
fix virtual functions and serialization
divyegala 51227b1
invert conditional
divyegala 579a301
debug msg
divyegala ae0b8ba
more debug prints
divyegala 4170a3e
fix efSearch setting in base search
divyegala 75808b1
re-negate ip distances in search_level
divyegala 09fb95b
Merge remote-tracking branch 'upstream/main' into raft-cagra-hnsw
divyegala 9bd1039
fix format
divyegala ea8028d
re-up minimum recall for base only IP distance
divyegala fc31351
add python tests
divyegala 3e2d343
Merge remote-tracking branch 'upstream/main' into raft-cagra-hnsw
divyegala 03ee1fb
ifdef guards in gpu cloner
divyegala 2e9cbc8
option to exclude dataset store on index
divyegala 826ed98
sys.big_endian to sys.byteorder (#3422)
kuarora 4dcdff5
Adding buck target for experiment bench_fw_ivf (#3423)
kuarora 756ad46
add skip_storage flag to HNSW (#3487)
mdouze c48fba3
Add cpp tutorial for index factory refine index construction (#3494)
66fd9ac
Update .gitignore (#3492)
abhiramvad 4615b00
rebase on latest changes
tarang-jain 2ce4b28
fix spurious include to land the cagra diff (#3502)
mdouze 959cd49
add use_raft to knn_gpu (torch) (#3509)
algoriddle 2d1e49a
Add conda bin to path early in the cmake GitHub action (#3512)
ramilbakhshyiev 11d6ce4
typo in test_io_no_storage (#3515)
algoriddle a651593
Consolidate build environment configuration steps in cmake builds (#3…
ramilbakhshyiev 6d7317d
fix Windows build - signed int OMP for MSVC (#3517)
algoriddle 1e0e35e
Unbreak RAFT conda builds (#3519)
ramilbakhshyiev 5109d96
Bump libraft to 24.06 to unblock nightly RAFT builds (#3522)
ramilbakhshyiev 4e98f8f
Add ABS_INNER_PRODUCT metric (#3524)
mdouze bea7343
initial commit
tarang-jain 45c4d49
Adding faiss bench_fw to bento faiss kernel (#3531)
kuarora c9e23ef
Refactor bench_fw to support train, build & search in parallel (#3527)
kuarora 8d24404
Add SQ8bit signed quantization (#3501)
naveentatikonda 702c3d2
rm build.sh
tarang-jain a918768
remove compilation issues
tarang-jain fb628eb
pull upstream
tarang-jain 499cae4
include common header; change dependency to libcuvs
tarang-jain ca45475
variable names changed
tarang-jain 14e830d
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 45c42bb
ivf-pq tests pass
tarang-jain c89b8f3
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain a0a9b40
update caps
tarang-jain 0106e12
fix failing tests
tarang-jain b7943de
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 57915d4
update CMake, tests
tarang-jain 3393d33
update github actions; conda pkg
tarang-jain 87e2e6c
formatting; update github actions
tarang-jain 5dad218
format
tarang-jain 245fe20
small change to error message
tarang-jain 1fe434a
don't need to link against raft::raft
b63d543
rm build.sh
6edead6
Merge branch 'main' into cuvs-migrate
tarang-jain 2d9ff7d
merge latest main
tarang-jain a598ecb
cleanup docs; merge latest main
tarang-jain 5794259
Merge branch 'main' into cuvs-migrate
tarang-jain bd01b85
Merge branch 'main' into cuvs-migrate
tarang-jain 8f5a065
merge latest main
f62ed13
rm RAFT mentions
171979e
update action.yml
68b3d2d
undo some merge changes
bd80b31
undo some merge changes
a9efc3f
updates after PR reviews
90ccb63
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
8d4496a
rename test_raft.py
3deb1e1
resolve failing Cagra test
4fdea22
clang-format
bb9bd67
Merge branch 'main' into cuvs-migrate
tarang-jain f723d8c
fix compilation issues
tarang-jain 74d1fa5
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
5f61dab
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
0be00b8
Merge branch 'main' into cuvs-migrate
tarang-jain c6da21c
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
74e21ab
resolve bfknn test failures
tarang-jain 4feff2a
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 3e097ce
Delete build.sh
tarang-jain 15f4218
Merge branch 'main' into cuvs-migrate
tarang-jain 141bcb9
merge upstream main
tarang-jain 337a74a
Apply suggestions from code review
tarang-jain 62bf7f3
update stream; dont reset handle
tarang-jain c75a7cc
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
tarang-jain aae6cf2
resolve compilation error
tarang-jain 4b84f46
do not link cutlass
tarang-jain 8bd4793
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain a6f1775
link raft::raft
tarang-jain 3ccb6d6
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain c1b959c
sconditionally fndinng raft
tarang-jain 7024c0d
Trigger Build
tarang-jain f207def
change link order
tarang-jain 42dc0b7
Merge branch 'main' into cuvs-migrate
tarang-jain 7ac798d
endif()
tarang-jain 64d424a
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain e10a05e
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
tarang-jain 93172cf
change installed libcuvs version in git actions
tarang-jain 1b183c9
Merge branch 'main' into cuvs-migrate
tarang-jain 2691b08
Merge branch 'main' into cuvs-migrate
tarang-jain d4851ab
Merge branch 'main' into cuvs-migrate
tarang-jain 54df8fa
Merge branch 'main' into cuvs-migrate
tarang-jain dbde0f3
dbg test, update pinned cuvs
tarang-jain 3c775f7
update action.yml
tarang-jain 07f8945
merge main
tarang-jain 1d10929
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
tarang-jain a179697
rm debug statements, restore torch tests
tarang-jain 5d998da
Merge branch 'main' into cuvs-migrate
tarang-jain 5509c49
remove find_package(raft)
tarang-jain c0629c6
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 43f9c3f
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
tarang-jain 4a2228c
make brute_force::search compatible with 24.10
tarang-jain d55c553
rm raft::raft linkage
tarang-jain ff1555c
rm merge conflict
tarang-jain 31c91e6
downgrade cmake version
tarang-jain 16e23bb
rm mentions of use_raft
tarang-jain f472eb4
empty commit
tarang-jain 01bca71
Build CMake from source from v3.30.5
KyleFromNVIDIA 10344a1
do not symblink sys dependencies for cuvs
tarang-jain b28fd89
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain b1bb9ba
update torch test
tarang-jain 6573614
Use build tree CMake instead of attempting to install
KyleFromNVIDIA 3229a38
revert
tarang-jain 4380cd4
Configure without OpenSSL
KyleFromNVIDIA f5bb016
Install and use OpenSSL
KyleFromNVIDIA 8474325
Set CMAKE_PREFIX_PATH
KyleFromNVIDIA 0ed617d
Merge branch 'main' into cmake-bisect
KyleFromNVIDIA 3eb6a56
glibc version
tarang-jain 80bb89c
merge upstream
tarang-jain abfe7f4
install cmd
tarang-jain 6a8cea0
Fix merge conflict
KyleFromNVIDIA 26a45f2
upgrade pkgs
tarang-jain 00475bc
Use CMAKE_FIND_ROOT_PATH
KyleFromNVIDIA ba2f399
Debug find
KyleFromNVIDIA 0706be0
Use conda cmake to build cmake
KyleFromNVIDIA ed924f4
Remove leftover --
KyleFromNVIDIA ece94aa
Force OpenSSL directory
KyleFromNVIDIA b46d70d
OPENSSL_ROOT_DIR
KyleFromNVIDIA 265fc52
link omp in tests
tarang-jain d9ebbab
debug find
KyleFromNVIDIA 209048a
Xcompiler
tarang-jain f045e8d
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
54e772c
add raft::raft to tests cmake
tarang-jain 20b19df
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
e0d5ff9
Use CMAKE_PREFIX_PATH
KyleFromNVIDIA 4ba1389
update compile_options
tarang-jain 5d82598
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 19ddf28
downgrade to cuvs=24.08
tarang-jain 0ab074a
action
tarang-jain fa6365b
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 81ecbb8
Activate Conda environment for CMake build
KyleFromNVIDIA cac235e
Merge branch 'main' into cmake-bisect
KyleFromNVIDIA 69f9ea8
conda init
KyleFromNVIDIA 6f7fecc
Init current shell
KyleFromNVIDIA 140e5e1
source activate
KyleFromNVIDIA 7ad8315
upgrade cmake with libcuvs=24.08
tarang-jain 4e67ec0
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain f3fb559
Merge branch 'main' into cuvs-migrate
tarang-jain 7d52e98
merge upstream; add FIXME; rm raft::raft linkage
tarang-jain c23df0a
one more try with 24.10
tarang-jain 468335f
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
tarang-jain b4989e6
one more try with 24.10
tarang-jain bb5cd83
rm std::nullopt
tarang-jain 3361b44
revert changes to should_use_cuvs
tarang-jain 2364650
Use Ubuntu libssl-dev instead of Conda
KyleFromNVIDIA a7036bd
update
KyleFromNVIDIA 673a7fb
Don't install openssl in Conda
KyleFromNVIDIA 3b8305f
Merge branch 'cuvs-migrate' into cmake-bisect
KyleFromNVIDIA 461da6f
Merge branch 'main' into cmake-bisect
KyleFromNVIDIA c23bc3b
Bootstrap again
KyleFromNVIDIA 12a3a6f
Show libc6
KyleFromNVIDIA 60cf7c3
apt-cache
KyleFromNVIDIA 7568e7a
Disable symlink step
KyleFromNVIDIA a9e6c13
Verbose makefile
KyleFromNVIDIA cba2c6c
Merge branch 'main' into cmake-bisect
KyleFromNVIDIA 65c286a
Link against Threads::Threads
KyleFromNVIDIA 4decfb4
pin gcc_linux-64 and sysroot_linux-64
tarang-jain 6bbe302
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 929a6d8
do not symlink system deps
tarang-jain 48ba59d
put conda-forge before nvidia channel
tarang-jain 04f32f9
merge upstream
tarang-jain d8ebaa3
set device scope in cagra train
tarang-jain 06aace2
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain d779cf2
unpin pkgs in action
tarang-jain 38e14b4
pin cmake
tarang-jain ca7ed65
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
9de6563
specify conda-forge while install base deps
tarang-jain 5577a4a
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 0d0d7d3
install cmake from conda-forge
tarang-jain 2bbdd15
rm conda-forge
tarang-jain 281c2b6
only install cmake from conda-forge
tarang-jain bb68cf7
make from conda-forge
tarang-jain a8c2926
revert
tarang-jain b821972
debug failing cagra test
tarang-jain dd16e77
revert to 24.08
tarang-jain 28ea578
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
tarang-jain 836a476
revert to 24.08
tarang-jain aa6e260
style
tarang-jain 32bb62b
copyright
tarang-jain aa64371
StandardGpuResources
tarang-jain 406dd79
Merge branch 'cuvs-migrate' of https://github.com/tarang-jain/faiss i…
af3bfcc
apply all changes to standardgpuresources
3fb67cc
rm build.sh
3e056ed
Merge branch 'main' of https://github.com/facebookresearch/faiss into…
3e40755
Try using CMake 3.29
KyleFromNVIDIA 1e81f5d
Merge branch 'cuvs-migrate' into cmake-bisect
KyleFromNVIDIA 6380a95
Fix newline
KyleFromNVIDIA bcabd69
Fix commenting
KyleFromNVIDIA 24827c9
Fix CMake version
KyleFromNVIDIA 7faf767
Try CMake 3.28
KyleFromNVIDIA 031a50c
Try CMake 3.27
KyleFromNVIDIA 11e4999
Try CMake 3.26
KyleFromNVIDIA b753f59
Try CMake 3.27 from conda-forge
KyleFromNVIDIA aab1913
Try CMake 3.30 from conda-forge
KyleFromNVIDIA e6816a3
Try CMake 3.28 from conda-forge
KyleFromNVIDIA 53af574
Try CMake 3.29 from conda-forge
KyleFromNVIDIA File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.