-
Notifications
You must be signed in to change notification settings - Fork 15
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
Move stdexec compilation to c++ files + add CI for cuda stdexec configuration #1188
Conversation
aurianer
commented
Sep 16, 2024
•
edited
Loading
edited
- Move stdexec compilation out of cuda files.
- Remove spack conflict when cuda and pika cxxstd=20.
- Attempt to add CI with gcc 12, cuda 11 and stdexec.
- Use @msimberg's fix for latest stdexec, I'll rebased when Reenable stdexec GCC CI configurations #1187 is merged.
69a4d67
to
b32c9e4
Compare
cscs-ci run |
I opened #1189 to reenable testing once CI is on alps |
cscs-ci run |
bc2f685
to
a3ee87d
Compare
cscs-ci run |
5d70660
to
2ee79f4
Compare
cscs-ci run |
Four(!) hangs on the hermitian matrix multiplication test in different configurations this time. I suppose it's time to try to reproduce it again. Clearly something is still badly wrong there. (Two of the hangs were after the test itself has passed, two in the middle of the test.) Unrelated to the changes here obviously... |
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.
Minor comments. This looks good to me otherwise. Thanks @aurianer!
2ee79f4
to
4c74b96
Compare
cscs-ci run |
CUDA@12 doesn't work on daint (unsufficient drivers)
4c74b96
to
f1a43df
Compare
cscs-ci run |
Only thing that came to mind now is that with this change there are three GCC 13 CI configurations. Do we want to prune some away or keep them all? |