-
Notifications
You must be signed in to change notification settings - Fork 532
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
[REVIEW] AgglomerativeClustering support single cluster and ignore only zero distances from self-loops #3824
[REVIEW] AgglomerativeClustering support single cluster and ignore only zero distances from self-loops #3824
Conversation
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.
Changes look good pending merging of the PR in RAFT and updating the commit hash once that's merged
rerun tests |
argh the CI test failure was unfortunate timing that cause mismatched cuDF and libcudf:
|
rerun tests |
@gpucibot merge |
Codecov Report
@@ Coverage Diff @@
## branch-21.06 #3824 +/- ##
===============================================
Coverage ? 85.41%
===============================================
Files ? 227
Lines ? 17313
Branches ? 0
===============================================
Hits ? 14788
Misses ? 2525
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
…istances from self-loops (rapidsai#3824) Closes rapidsai#3801 Closes rapidsai#3802 Corresponding RAFT PR: rapidsai/raft#217 Authors: - Corey J. Nolet (https://github.com/cjnolet) Approvers: - Dante Gama Dessavre (https://github.com/dantegd) URL: rapidsai#3824
Closes #3801
Closes #3802
Corresponding RAFT PR: rapidsai/raft#217