Skip to content

Fix building cuml with CCCL main#5886

Merged
rapids-bot[bot] merged 21 commits intorapidsai:branch-24.06from
trxcllnt:fix/cccl-2.5
May 29, 2024
Merged

Fix building cuml with CCCL main#5886
rapids-bot[bot] merged 21 commits intorapidsai:branch-24.06from
trxcllnt:fix/cccl-2.5

Conversation

@trxcllnt
Copy link
Copy Markdown
Collaborator

@trxcllnt trxcllnt commented May 8, 2024

Similar to rapidsai/cudf#15552, we are testing building RAPIDS with CCCL's main branch to get ahead of any breaking changes.

@trxcllnt trxcllnt requested a review from a team as a code owner May 8, 2024 21:53
@trxcllnt trxcllnt added DO NOT MERGE Hold off on merging; see PR for details improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 8, 2024
@trxcllnt trxcllnt requested a review from a team as a code owner May 8, 2024 22:07
@github-actions github-actions bot added the CMake label May 8, 2024
@trxcllnt trxcllnt requested a review from a team as a code owner May 9, 2024 20:30
@trxcllnt trxcllnt added 3 - Ready for Review Ready for review by team and removed DO NOT MERGE Hold off on merging; see PR for details labels May 13, 2024
@trxcllnt
Copy link
Copy Markdown
Collaborator Author

Looks like the changes in this PR are working and the failing tests are due to missing RAFT symbols.

@dantegd
Copy link
Copy Markdown
Member

dantegd commented May 29, 2024

/merge

@rapids-bot rapids-bot bot merged commit 92f5830 into rapidsai:branch-24.06 May 29, 2024
rapids-bot bot pushed a commit to rapidsai/rapids-cmake that referenced this pull request Jun 7, 2024
Updates CCCL to v2.5.0 and a more recent cuCollections commit.

Using cuco at [6923b3](NVIDIA/cuCollections@6923b3b) because it was before NVIDIA/cuCollections#479, which is a breaking change for libcudf.

CCCL PR:
* NVIDIA/cccl#1667

RAPIDS PRs:
* rapidsai/cudf#15552
* rapidsai/cuml#5886
* rapidsai/cugraph#4404
* rapidsai/cuspatial#1382

Authors:
  - Paul Taylor (https://github.com/trxcllnt)
  - Vyas Ramasubramani (https://github.com/vyasr)

Approvers:
  - Robert Maynard (https://github.com/robertmaynard)
  - Bradley Dice (https://github.com/bdice)

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

Labels

3 - Ready for Review Ready for review by team 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.

4 participants