-
Notifications
You must be signed in to change notification settings - Fork 184
[Feat] Add support of logical merge in Cagra #713
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
Merged
rapids-bot
merged 42 commits into
rapidsai:branch-25.06
from
rhdong:rhdong/logical-merge
May 28, 2025
Merged
Changes from 39 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
cb8f16b
[Feat] Add support of logical merge in Cagra
rhdong 6c00e52
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong 450df2a
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong 4debc02
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong c5ecfe8
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong b3a99c6
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong bcfa656
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong 186be95
fix ci fail
rhdong 6138f92
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong b962830
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong 1e61e1b
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong 590c3a8
Merge branch 'branch-25.04' into rhdong/logical-merge
rhdong a986d2e
resolve conflicts
rhdong 73b96e3
Merge remote-tracking branch 'origin/branch-25.06' into rhdong/logica…
rhdong f357c6e
fix errors when processing conflicts & supports i64 output indices.
rhdong 37bbfe5
Merge branch 'rhdong/logical-merge-dev' into rhdong/logical-merge
rhdong 8caa251
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong e229aee
Merge remote-tracking branch 'origin/branch-25.06' into rhdong/logica…
rhdong 5076f8f
fix small dataset issue by avoid graph degree reduce to 1
rhdong 7610e65
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong 0f42309
Merge remote-tracking branch 'origin/branch-25.06' into rhdong/logica…
rhdong acb4704
process conflict
rhdong 63e9d55
Merge branch 'rhdong/logical-merge-dev' into rhdong/logical-merge
rhdong db0c624
introduce raft::copy for 2Dcpy
rhdong c1023e1
TBR: pin the raft for temp CI verify.
rhdong 39e7938
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong 9e112a7
Revert "TBR: pin the raft for temp CI verify."
rhdong f5eb6e4
replace merge_top_k_gpu by select_k
rhdong da8c2e5
add benchmark for CAGRA merge
rhdong 0342358
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong ddef8ce
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong 147be54
Refactor composite wrappers and merge API with redesigned index system
rhdong acbcb50
add back default constructor
rhdong 82d978b
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong 007b9da
remove MergeStrategy reference of C header
rhdong 9a69747
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong e19590d
add support multi-stream for CompositeIndex::search
rhdong eca7283
setup stream number for resource handle by using set_cuda_stream_pool
rhdong bf0bca5
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong 7d49411
Merge branch 'branch-25.06' into rhdong/logical-merge
rhdong f2ec31c
set n_stream equal to the size of sub_indices
rhdong d08f458
optimize naming for split num.
rhdong 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
out of curiosity, why 8 here? I don't think we need to hold up the release for this, but can you at least create a Github issue to make this configurable (or to find a good reasonable default which can be overridden)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just changed it to be equal to
wrapped_indices.size()