-
Notifications
You must be signed in to change notification settings - Fork 576
UMAP knn build with cuVS all-neighbors #6563
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
Conversation
cpp/cmake/thirdparty/get_cuvs.cmake
Outdated
@@ -48,8 +48,8 @@ function(find_and_configure_cuvs) | |||
BUILD_EXPORT_SET cuml-exports | |||
INSTALL_EXPORT_SET cuml-exports | |||
CPM_ARGS | |||
GIT_REPOSITORY https://github.com/${PKG_FORK}/cuvs.git | |||
GIT_TAG ${PKG_PINNED_TAG} | |||
GIT_REPOSITORY https://github.com/jinsolp/cuvs.git |
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.
Needs to be reverted once this PR in cuvs is merged
/ok to test |
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Description
Replacing usage of raft NN Descent to cuVS all-neighbors wrapper (which internally uses NN Descent and includes the batching feature). Planning to add some interesting numbers to this PR...
Usage