Skip to content

Revert precomputed kNN on CPU for UMAP#7576

Merged
rockhowse merged 1 commit intorapidsai:release/25.12from
jinsolp:revert-host-knn
Dec 9, 2025
Merged

Revert precomputed kNN on CPU for UMAP#7576
rockhowse merged 1 commit intorapidsai:release/25.12from
jinsolp:revert-host-knn

Conversation

@jinsolp
Copy link
Copy Markdown
Contributor

@jinsolp jinsolp commented Dec 9, 2025

Reverting this PR: #7481 because it needs further testing on pointer residency on HMM-configure machines.

Related issue: #7540

@jinsolp jinsolp self-assigned this Dec 9, 2025
@jinsolp jinsolp requested review from a team as code owners December 9, 2025 00:14
@github-actions github-actions Bot added Cython / Python Cython or Python issue CUDA/C++ labels Dec 9, 2025
@jinsolp jinsolp added bug Something isn't working non-breaking Non-breaking change and removed Cython / Python Cython or Python issue CUDA/C++ labels Dec 9, 2025
@jinsolp
Copy link
Copy Markdown
Contributor Author

jinsolp commented Dec 9, 2025

Reopen this issue if this PR is merged: #7143

Copy link
Copy Markdown
Member

@aamijar aamijar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jinsolp
Copy link
Copy Markdown
Contributor Author

jinsolp commented Dec 9, 2025

@aamijar No that just exposes an option and will still work as before because the default is mem_type="device" 🙂

@rockhowse rockhowse merged commit f8a3f01 into rapidsai:release/25.12 Dec 9, 2025
112 checks passed
mani-builds pushed a commit to mani-builds/cuml that referenced this pull request Jan 11, 2026
Admin merging per build-eng request

Reverting this PR: rapidsai#7481 because it
needs further testing on pointer residency on HMM-configure machines.

Related issue: rapidsai#7540
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants