Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use RMM's patched CCCL (2.1.x) #11353

Open
wants to merge 1 commit into
base: release_2.1.0
Choose a base branch
from

Conversation

jakirkham
Copy link
Contributor

Backports to XGBoost 2.1.x PR: #11351

Make sure to search for RMM if it will be used. This should pick up the patched CCCL from RMM.

If RMM is not being used and this is a CUDA build, search for CCCL explicitly.

Make sure to search for RMM if it will be used. This should pick up the
patched CCCL from RMM.

If RMM is not being used and this is a CUDA build, search for CCCL
explicitly.
@jakirkham
Copy link
Contributor Author

Using this as the basis for a patch in RAPIDS

If we do not plan any additional XGBoost 2.1.x releases, then this can just stay open for now

@jakirkham
Copy link
Contributor Author

On CI am seeing errors like this one:

Download action repository 'actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11' (SHA:b4ffde65f46336ab88eb53be808477a3936bae11)
Download action repository 'actions/setup-java@99b8673ff64fbf99d8d325f52d9a5bdedb8483e9' (SHA:99b8673ff64fbf99d8d325f52d9a5bdedb8483e9)
Download action repository 'conda-incubator/setup-miniconda@a4260408e20b96e80095f42ff7f1a15b27dd94ca' (SHA:a4260408e20b96e80095f42ff7f1a15b27dd94ca)
Error: Missing download info for actions/cache@13aacd865c20de90d75de3b17ebe84f7a17d57d2

IOW it appears to be running into some issue fetching GitHub Actions needed for workflow steps

@trivialfis trivialfis changed the title Use RMM's pached CCCL (2.1.x) Use RMM's patched CCCL (2.1.x) Mar 20, 2025
@hcho3
Copy link
Collaborator

hcho3 commented Mar 26, 2025

@jakirkham Do we still need this pull request, now that rapidsai/xgboost-feedstock#85 is merged?

@jakirkham
Copy link
Contributor Author

Let's leave this here for now. Looks like we may still need this a bit longer for XGBoost CUDA 11 builds

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants