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 cuda.bindings layout. #6233

Merged
merged 3 commits into from
Jan 24, 2025
Merged

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Jan 17, 2025

This PR updates cuML to use the new cuda-python cuda.bindings layout. See rapidsai/build-planning#117.

@bdice bdice requested a review from a team as a code owner January 17, 2025 14:12
@bdice bdice requested review from dantegd and viclafargue January 17, 2025 14:12
@github-actions github-actions bot added the Cython / Python Cython or Python issue label Jan 17, 2025
@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jan 17, 2025
@bdice bdice self-assigned this Jan 17, 2025
@jakirkham
Copy link
Member

A couple CI jobs were failing and this PR was a little out-of-date. Likely these are issues that have been fixed more recently. So updated the PR

@bdice
Copy link
Contributor Author

bdice commented Jan 23, 2025

/merge

@jakirkham
Copy link
Member

Looks like the CUDA 11.8 wheel tests still fail. Did we find a resolution to that?

@bdice
Copy link
Contributor Author

bdice commented Jan 23, 2025

Looks like the CUDA 11.8 wheel tests still fail. Did we find a resolution to that?

Yes, see rapidsai/build-planning#137. The PR to RAFT rapidsai/raft#2548 fixed it. These failures are not the same.

@jakirkham
Copy link
Member

Though it looks like that was merged 2 days ago. It looked like the failure here was more recent. We have also been seeing other CI issues with cuML. Will add you to a thread

@rapids-bot rapids-bot bot merged commit b1ac238 into rapidsai:branch-25.02 Jan 24, 2025
62 checks passed
@jakirkham
Copy link
Member

Huzzah! 🎉

The last CI restart fixed it 😄

Thanks Bradley and Dante! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cython / Python Cython or Python issue 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.

3 participants