Skip to content
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

Forward-merge branch-24.10 into branch-24.12 #384

Merged
merged 1 commit into from
Oct 3, 2024
Merged

Conversation

rapids-bot[bot]
Copy link

@rapids-bot rapids-bot bot commented Oct 3, 2024

Forward-merge triggered by push to branch-24.10 that creates a PR to keep branch-24.12 up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge. See forward-merger docs for more info.

Adds Vamana (DiskANN index construction support. Includes all files and changes to expose the vamana graph index in the C++ API, **build** and **serialize** operations only.  Build runs the iterative Vamana index construction, and serialize writes the graph to a file that matches the format of the Microsoft DiskANN library. This lets users use cuVS for index construction while still doing search with their current infrastructure.  Search support to be added at a later time.

Some items missing / not included in the PR:

- Documentation
- Python and C API support
- Tests that verify recall (can't do this without search)
- Additional distance metrics (only L2 currently)

Current known issues / bugs:
- Odd dimension datasets not supported (causes alignment error)
- Memory footprint is still large, need some batching / alg changes to improve
- Adding a more sophisticated medoid selection (minor recall benefit)
- Improving shared memory usage for high-degree construction should improve perf

Authors:
  - Ben Karsin (https://github.com/bkarsin)
  - Tarang Jain (https://github.com/tarang-jain)
  - Corey J. Nolet (https://github.com/cjnolet)
  - Dante Gama Dessavre (https://github.com/dantegd)

Approvers:
  - Tamas Bela Feher (https://github.com/tfeher)
  - Corey J. Nolet (https://github.com/cjnolet)
  - Tarang Jain (https://github.com/tarang-jain)

URL: #339
@rapids-bot rapids-bot bot requested review from a team as code owners October 3, 2024 02:05
@GPUtester GPUtester merged commit aeacfdf into branch-24.12 Oct 3, 2024
Copy link
Author

rapids-bot bot commented Oct 3, 2024

SUCCESS - forward-merge complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants