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

Reenable pytest cuml-dask for CUDA 12.5 wheel CI tests #6051

Merged
merged 7 commits into from
Sep 18, 2024

Conversation

divyegala
Copy link
Member

Reference issue: #6050

@divyegala divyegala added bug Something isn't working non-breaking Non-breaking change labels Aug 28, 2024
@divyegala divyegala requested a review from a team as a code owner August 28, 2024 08:27
@github-actions github-actions bot added the ci label Aug 28, 2024
@jakirkham
Copy link
Member

Seeing some of the tests failing due to NumPy 2 compatibility issues on CI. This is the same thing we have seen elsewhere: #6047 (comment)

Think the reason we are getting NumPy 2 here is there is not a constraint on this dependency in cuML

We are fixing both of these issues in PR: #6031

Have taken the changes in this PR and merged them into PR: #6031

That way we have all the CI fixes and skips in one place so we can hopefully clear out the CI issues in one go

@jakirkham
Copy link
Member

Thank you Divye for putting this and issue ( #6050 ) together! 🙏

@jakirkham jakirkham mentioned this pull request Aug 28, 2024
@jakirkham jakirkham changed the title Skip pytest cuml-dask for CUDA 12.5 wheel CI tests Reenable pytest cuml-dask for CUDA 12.5 wheel CI tests Aug 28, 2024
@jakirkham
Copy link
Member

Have tweaked this to reenable the skipped wheel tests so we have somewhere to test and fix this issue

Hope that is ok 🙂

@divyegala
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit 7de8831 into rapidsai:branch-24.10 Sep 18, 2024
61 of 62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants