-
Notifications
You must be signed in to change notification settings - Fork 128
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
ci: use nvcc_wrapper in adiosKokkos #3623
ci: use nvcc_wrapper in adiosKokkos #3623
Conversation
7cfe742
to
b893ba2
Compare
bc39c5a
to
1b8541e
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.
Thanks Vicente for working on this. I will test this on Summit/Perlmutter/Crusher and come back to update the review
1b8541e
to
e018ff5
Compare
e018ff5
to
cc60961
Compare
@anagainaru fantastic work there checking where it builds/runs 👍 👍 👍 |
I realized that now we can always use |
…rapper ci: use nvcc_wrapper in adiosKokkos (cherry picked from commit 189f384)
* release_29: Allow Span in files opened for Append crusher,ci: set unique env per pipeline fixed #3638 CI,windows: change MSMPI URL Bump requests from 2.28.1 to 2.31.0 in /docs docs: update whatsnew ci,crusher: minor tweaks ci,olcf,crusher: enable Crusher CI ci,ascent: enable ascent builds Merge pull request #3623 from vicentebolea/kokkos-use-nvcc-wrapper BP5: fixes memory error with IBM XL
fixes #3566
Taken from the comment added in the changes