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

Update for RAPIDS C++ wheels #138

Closed
1 of 2 tasks
vyasr opened this issue May 3, 2024 · 3 comments · Fixed by #215
Closed
1 of 2 tasks

Update for RAPIDS C++ wheels #138

vyasr opened this issue May 3, 2024 · 3 comments · Fixed by #215
Assignees

Comments

@vyasr
Copy link
Contributor

vyasr commented May 3, 2024

Once rapidsai/build-planning#33 is complete, we will have a new set of wheels that change some of the requirements of our CI images. This issue is to keep track of such changes as we discover them so that we can make them all at the end.

  • We will no longer need to install UCX into the images since UCX will be available as a wheel.
  • We will need to install the wheel Python package in addition to auditwheel because for the header-only C++ libraries the wheels won't contain any binaries, so auditwheel will be a no-op (auditwheel repair will not retag wheels that have no binaries to repair).
  • add 'wheel' to ci-wheel image #215
@jameslamb
Copy link
Member

We will no longer need to install UCX into the images since UCX will be available as a wheel.

Just noting this is done: #154

I've updated the description of this issue.

@bdice
Copy link
Contributor

bdice commented Aug 27, 2024

@vyasr Is there still work needed here? It seems like we have shipped librmm and libcudf wheels already so I am questioning what actions are needed, if any.

@vyasr
Copy link
Contributor Author

vyasr commented Aug 27, 2024

We need to add wheel to the list of dependencies to install here. After that, I'm happy to close this issue and track other changes in one-off issues if you'd prefer.

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 a pull request may close this issue.

3 participants