-
Notifications
You must be signed in to change notification settings - Fork 706
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
{mpi}[NVHPC/24.1-CUDA-12.4.0] OpenMPI v4.1.6 w/ CUDA 12.3.0 #21566
base: develop
Are you sure you want to change the base?
{mpi}[NVHPC/24.1-CUDA-12.4.0] OpenMPI v4.1.6 w/ CUDA 12.3.0 #21566
Conversation
easybuild/easyconfigs/o/OpenMPI/OpenMPI-4.1.6-NVHPC-24.1-CUDA-12.4.0-CUDAcore-12.4.0.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/o/OpenMPI/OpenMPI-4.1.6-NVHPC-24.1-CUDA-12.4.0-CUDAcore-12.4.0.eb
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,72 @@ | |||
name = 'OpenMPI' | |||
version = '4.1.6' | |||
versionsuffix = '-CUDAcore-12.4.0' |
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.
Is there any specific reason for CUDAcore
?
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.
No, I guess that was a typo from my side.
I have addressed your comments above, thanks for checking! |
@Thyre: Any updates on this, please? |
The filename should be Aside from that the PR looks fine to me. However, I'm not able to approve or merge this PR. In addition, we should at least do some testing. I hopefully can check on my machines on Friday. |
…nMPI-4.1.6-NVHPC-24.1-CUDA-12.4.0.eb
NVHPC/24.1-CUDA-12.4.0 does not exist, but NVHPC/24.1-CUDA-12.3.0 does. It's probably the easiest solution to switch to CUDA 12.3.0. |
…-24.1-CUDA-12.3.0.eb
I changed it to CUDA-12.3.0. But |
Well that's annoying... |
@Thyre But EB already have UCX-CUDA based on CUDA-12.4.0 (link). I am bit confused now! Do I need to write UCX-CUDA-1.15.0 based on CUDA-12.3.0 and same for UCC-CUDA? Same for NCCL (which is a dependency of UCC-CUDA) where EB has it with CUDA-12.4.0 (link). |
(created using
eb --new-pr
)