8000 Support passing a dataset to `FromCagra` by ajit283 · Pull Request #767 · rapidsai/cuvs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Support passing a dataset to FromCagra #767

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
merged 7 commits into from
Mar 22, 2025

Conversation

ajit283
Copy link
Contributor
@ajit283 ajit283 commented Mar 15, 2025

For out-of-core CAGRA index builds where the dataset does not fit on GPU memory, a host tensor can be passed using this.

@ajit283 ajit283 requested a review from a team as a code owner March 15, 2025 07:43
Copy link
copy-pr-bot bot commented Mar 15, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions bot added the cpp label Mar 15, 2025
@divyegala
Copy link
Member

This PR looks good to me but is there a reason we are attempting to merge it to branch-25.02? @cjnolet is this supposed to be a backport?

@cjnolet
Copy link
Member
cjnolet commented Mar 15, 2025

Nope, we need to carry this forward to 25.04. No more features can be merged into 25.02. Thanks for reviewing @divyegala!

@divyegala
Copy link
Member

@ajit283 please re-target to 25.04 and I will approve this PR

@ajit283 ajit283 changed the base branch from branch-25.02 to branch-25.04 March 15, 2025 20:15
@cjnolet cjnolet added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Mar 15, 2025
@cjnolet
Copy link
Member
cjnolet commented Mar 15, 2025

/ok to test

< 8000 /div>
@divyegala
Copy link
Member

/ok to test

@divyegala
Copy link
Member

/ok to test

@divyegala
Copy link
Member

/ok to test

@divyegala
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 813c946 into rapidsai:branch-25.04 Mar 22, 2025
66 checks passed
jiangyinzuo pushed a commit to jiangyinzuo/cuvs that referenced this pull request May 25, 2025
For out-of-core CAGRA index builds where the dataset does not fit on GPU memory, a host tensor can be passed using this.

Authors:
  - Ajit Mistry (https://github.com/ajit283)
  - Corey J. Nolet (https://github.com/cjnolet<
7BF1
/a>)
  - Divye Gala (https://github.com/divyegala)

Approvers:
  - Divye Gala (https://github.com/divyegala)

URL: rapidsai#767
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Development

Successfully merging this pull request may close these issues.

3 participants
0