-
Notifications
You must be signed in to change notification settings - Fork 41
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
Require Kokkos 3.7 #834
Require Kokkos 3.7 #834
Conversation
97c5a03
to
0eb0612
Compare
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.
Looks OK to me.
Did you mean to move the majority of builds to 4.0? |
Yes. There are now 4 Kokkos 4.0 builds, and 3 Kokkos 3.7 builds. I think that's a reasonable split. |
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.
Kokkos 4 requires a minimum ROCm version of 5.2
So you'd need to update the base image
HIP fails with
Updated HIP container to use 5.2. Could go higher, to 5.3.3 or 5.4.2 if we want. |
d482453
to
c0663cd
Compare
HIP 5.2.0 error:
Corresponding code: ArborX/test/tstQueryTreeRay.cpp Line 146 in 0136f96
|
ed6a4e7
to
0910862
Compare
HIP 5.4.2 passes. |
HIP 5.3.3 also works. Merging. |
Merged with different builds passing at certain points (but not all together). |
Updates Kokkos versions in several builds:
HIP-5.0
GCC-12.2
Clang
CUDA-11.5.2-NVCC-CUDA-AWARE-MPI
CUDA-11.5.2-NVCC