-
Notifications
You must be signed in to change notification settings - Fork 50
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
Build CUDA under host_injections
and make EESSI aware of host CUDA drivers
#368
Build CUDA under host_injections
and make EESSI aware of host CUDA drivers
#368
Conversation
Instance
|
This can be tested by checking out the branch, starting the build container from the PR and running, e.g.,
|
But I need to transition the PR to (also?) use |
This PR can be tested without GPUs, use
to start the build container with the necessary settings, then you can install CUDA (under
@trz42 There are some sizable changes to the build container in the PR so you may wish to take it for a test drive. |
@trz42 I need some advice here. I want to probably always trigger the ability to install CUDA, but I only want to trigger the |
@casparvl I've addressed your review as much as I can and I've added better command line options, including a critical acceptance of the CUDA EULA. This should be read for showtime now |
I think I'll fold the script to the linking to the host drivers in here as well...working on it now |
host_injections
host_injections
and make EESSI aware of host CUDA drivers
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.
Lgtm! Awaiting rerun of a final CI test that crapped out because of rate limiting.
…4.1-gompi/2023a {2023.06}[gompi/2023a] BLAST+ V2.14.1
No description provided.