Skip to content

Conversation

@viclafargue
Copy link
Contributor

No description provided.

@aamijar
Copy link
Member

aamijar commented Dec 8, 2025

We are going to remove this file as part of the deprecated apis.
Can we use raft::memory_type_from_pointer instead?

@aamijar aamijar self-requested a review December 8, 2025 21:48
@cjnolet
Copy link
Member

cjnolet commented Jan 14, 2026

@viclafargue any thoughts about @aamijar's comment here?

@viclafargue
Copy link
Contributor Author

This PR was a temporary solution to handle special HMM-enabled GPU case that seemed to be causing issues in CI. But, I could unfortunately not reliably reproduce the issue on any machine (even HMM enabled ones) making this very difficult to debug and to know if this change actually solves the issue. For now we found a solution to avert the issue in cuML. I think it is better to close this PR for now. I will make sure to edit raft::memory_type_from_pointer instead if we decide to come back at it.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants