-
Notifications
You must be signed in to change notification settings - Fork 74
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
Port all conda recipes to rattler-build
#678
Conversation
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. |
/ok to test |
/ok to test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tacking on my changes from #656 into this PR. Please take these suggestions with a grain of salt -- I haven't fully validated them, and there may be some gray area here where we need to support libcufile on ARM and are willing to accept some minor issues on CUDA 12.0/12.1 in order to get the benefits of libcufile on CUDA 12.2+.
Co-authored-by: Bradley Dice <bdice@bradleydice.com>
This is the same failure I was seeing before:
On the arm job testing against cuda 12.0.1 So I think I'll revert the |
Hey @bdice -- I think I've got a reasonable compromise in-place here if you want to take a look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the specific concerns I had about libcufile on ARM. Sticking to the status quo for now and revisiting later seems like the best we can do.
/merge |
Port all condabuild recipes over to use
rattler-build
instead.Contributes to rapidsai/build-planning#47
rattler-build
, this changes all the licenses in thepyproject.toml
files to the SPDX-compliantApache-2.0
instead ofApache 2.0