Skip to content

Migrate to new CCCL memory resource interface#7450

Merged
rapids-bot[bot] merged 5 commits intorapidsai:mainfrom
bdice:branch-cuml-mr-migration
Nov 7, 2025
Merged

Migrate to new CCCL memory resource interface#7450
rapids-bot[bot] merged 5 commits intorapidsai:mainfrom
bdice:branch-cuml-mr-migration

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Nov 6, 2025

Migrates cuML to use the new CCCL memory resource interface.

Part of rapidsai/rmm#2126.

@bdice bdice requested a review from a team as a code owner November 6, 2025 04:05
@bdice bdice requested a review from divyegala November 6, 2025 04:05
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Nov 6, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

This PR migrates cuML to use the new stream-first parameter ordering for memory resource operations, replacing the legacy async methods:
- `allocate_async(size, alignment, stream)` → `allocate(stream, size, alignment)`
- `deallocate_async(ptr, size, alignment, stream)` → `deallocate(stream, ptr, size, alignment)`

Modified 9 files with 38 total API changes:
- ARIMA parameter management (14 changes)
- Genetic algorithm implementation (4 changes)
- SVM implementation and tests (12 changes)
- Pinned host vector (3 changes + added stream header)
- Benchmark infrastructure (2 changes)
- Symbolic regression example (2 changes)
- Sparse utilities (3 changes)

All changes maintain the same functionality while adopting the new CCCL interface.

Relates to rapidsai/rmm#2126
@bdice bdice force-pushed the branch-cuml-mr-migration branch from 0f0e1c5 to 5dcfd58 Compare November 6, 2025 04:08
@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 6, 2025
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Nov 6, 2025

/merge

@rapids-bot rapids-bot bot merged commit 75b0a5e into rapidsai:main Nov 7, 2025
187 of 202 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CUDA/C++ improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants