Skip to content

Adapt to rmm logger changes#6147

Merged
raydouglass merged 5 commits intorapidsai:branch-25.02from
vyasr:feat/rmm_logger
Dec 2, 2024
Merged

Adapt to rmm logger changes#6147
raydouglass merged 5 commits intorapidsai:branch-25.02from
vyasr:feat/rmm_logger

Conversation

@vyasr
Copy link
Copy Markdown
Contributor

@vyasr vyasr commented Nov 26, 2024

This PR adapts to breaking changes in rmm in rapidsai/rmm#1722.

@vyasr vyasr requested review from a team as code owners November 26, 2024 20:20
@github-actions github-actions Bot added conda conda issue Cython / Python Cython or Python issue CMake CUDA/C++ labels Nov 26, 2024
@betatim
Copy link
Copy Markdown
Member

betatim commented Nov 27, 2024

This PR includes a lot of changes that change 24.12 to 25.02 but it targets the 24.12 branch. Is that on purpose/for my education: why?

The actual change seems to be https://github.com/rapidsai/cuml/pull/6147/files#diff-1bba462ab050e89360fd88110a689e85ee037749cea091a1848ab574381d3795 which looks reasonable, though most of the CI jobs are unhappy so that needs investigating

@bdice bdice changed the base branch from branch-24.12 to branch-25.02 November 27, 2024 13:01
@bdice
Copy link
Copy Markdown
Contributor

bdice commented Nov 27, 2024

This needs to target 25.02, it’s just an oversight. I fixed that and merged the upstream. Hopefully we can fix this today before the Thanksgiving holiday. We are seeing breakage across multiple libraries.

@github-actions github-actions Bot removed conda conda issue Cython / Python Cython or Python issue labels Nov 27, 2024
@vyasr vyasr removed request for a team, raydouglass and teju85 November 27, 2024 22:59
Comment thread cpp/CMakeLists.txt Outdated
@vyasr vyasr added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 30, 2024
@raydouglass raydouglass merged commit 6f8a189 into rapidsai:branch-25.02 Dec 2, 2024
@vyasr vyasr deleted the feat/rmm_logger branch December 2, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake 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.

5 participants