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.

Sign up for GitHub

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

Jump to bottom

[Tracker] Pending items for NN Descent #419

Open

3 of 5 tasks

divyegala opened this issue Oct 21, 2024 · 0 comments

Assignees

Labels

improvement Python vector search

Member

divyegala commented Oct 21, 2024 •

edited

Loading

Consolidate code in cuVS by migrating over diff of features from RAFT
Triage CI wheel test failures
Triage batch test failures
Additional distances
Add mutual reachability API and triage HDBSCAN numerical instabilities when using NN Descent ([FEA] Enable HDBSCAN to build knn graph using NN Descent cuml#5939)

divyegala mentioned this issue

Migrate feature diff for NN Descent from RAFT to cuVS #421

Merged

cjnolet assigned divyegala

cjnolet added improvement vector search Python labels

cjnolet added this to VS/ML/DM Primitives Release Board

cjnolet moved this to In Progress in VS/ML/DM Primitives Release Board

rapids-bot bot pushed a commit that referenced this issue


          Migrate feature diff for NN Descent from RAFT to cuVS (#421)

29c0f5b

This PR is an amalgamation of the diff of 3 PRs in RAFT:

1. rapidsai/raft#2345
2. rapidsai/raft#2380
3. rapidsai/raft#2403

This PR also addresses part 1 and 2 of #419, closes #391 and makes CAGRA use the compiled headers of NN Descent, which seemed to have been a pending TODO https://github.com/rapidsai/cuvs/blob/009bb8de03ce9708d4d797166187250f77a59a36/cpp/src/neighbors/detail/cagra/cagra_build.cuh#L36-L37

Also, batch tests are disabled in this PR due to issue rapidsai/raft#2450. PR #424 will attempt to re-enable them.

Authors:
  - Divye Gala (https://github.com/divyegala)
  - Corey J. Nolet (https://github.com/cjnolet)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: #421

divyegala changed the title ~~Pending items for NN Descent~~ [Tracker] Pending items for NN Descent

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