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

[NO MRG] Test XGBoost 2.1.4 with CUDA 12.8 support #17947

Draft
wants to merge 2 commits into
base: branch-25.02
Choose a base branch
from

Conversation

jakirkham
Copy link
Member

Testing the recently produced packages of XGBoost 2.1.4 with CUDA 12.8 support.

xref: rapidsai/xgboost-feedstock#79

@jakirkham jakirkham requested a review from a team as a code owner February 7, 2025 05:35
@jakirkham jakirkham requested a review from raydouglass February 7, 2025 05:35
@jakirkham jakirkham marked this pull request as draft February 7, 2025 05:35
Copy link

copy-pr-bot bot commented Feb 7, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions bot added the Python Affects Python cuDF API. label Feb 7, 2025
@jakirkham jakirkham added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Feb 7, 2025
@jakirkham
Copy link
Member Author

/ok to test

@jakirkham jakirkham added the DO NOT MERGE Hold off on merging; see PR for details label Feb 7, 2025
@jakirkham jakirkham removed the request for review from raydouglass February 7, 2025 05:37
@jakirkham
Copy link
Member Author

This is a testing PR. No need to review

@github-actions github-actions bot added the cudf.pandas Issues specific to cudf.pandas label Feb 7, 2025
@jakirkham
Copy link
Member Author

/ok to test

@jakirkham
Copy link
Member Author

Not seeing much coverage of XGBoost in PR tests

It does look like nightlies test XGBoost

From a recent nightly using XGBoost 2.1.4, the tests appear to be passing. However the job fails likely due to warnings raised that are converted to errors

More details in issue: #17952

@bdice
Copy link
Contributor

bdice commented Feb 7, 2025

@jakirkham I think we should use the new xgboost conda packages here, so please consider turning this test PR into a mergeable PR. :)

@Matt711 or @galipremsagar may know why we were using pip wheels to begin with.

@Matt711
Copy link
Contributor

Matt711 commented Feb 7, 2025

@jakirkham I think we should use the new xgboost conda packages here, so please consider turning this test PR into a mergeable PR. :)

+1

@Matt711 or @galipremsagar may know why we were using pip wheels to begin with.

I think they were added before xgboost 2.0 was available on conda. See comment

@Matt711
Copy link
Contributor

Matt711 commented Feb 7, 2025

Not seeing much coverage of XGBoost in PR tests

It does look like nightlies test XGBoost

From a recent nightly using XGBoost 2.1.4, the tests appear to be passing. However the job fails likely due to warnings raised that are converted to errors

More details in issue: #17952

I think CI is actually failing due to Ibis dropping pandas support as a backend. I opened #17945 to resolve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cudf.pandas Issues specific to cudf.pandas DO NOT MERGE Hold off on merging; see PR for details improvement Improvement / enhancement to an existing function non-breaking Non-breaking change Python Affects Python cuDF API.
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants