Switch to CTest for CI and add NVHPC 23.5 to CI testing #56
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
reuterbal
force-pushed
the
nabr-nvidia-ci
branch
from
851bd85
to
ef46b1a
Compare
reuterbal
force-pushed
the
nabr-nvidia-ci
branch
from
ef46b1a
to
852c087
Compare
reuterbal
force-pushed
the
nabr-nvidia-ci
branch
3 times, most recently
from
fffdd81
to
bf5c149
Compare
reuterbal
force-pushed
the
nabr-nvidia-ci
branch
from
bf5c149
to
89cd907
Compare
reuterbal
changed the title
reuterbal
force-pushed
the
nabr-nvidia-ci
branch
10 times, most recently
from
c8e0273
to
0749717
Compare
reuterbal
force-pushed
the
nabr-nvidia-ci
branch
from
0749717
to
df29dc8
Compare
mlange05
approved these changes
Aug 9, 2023
| run: sudo apt-get install libopenmpi-dev | ||
|
|
||
| # Free up disk space for nvhpc | ||
| - name: Free Disk Space (Ubuntu) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Original intention was just to add a newer NVHPC version to Github CI but it went sideways pretty badly. Reason being that NVHPC seems to apply unsafe vectorization code paths that fail on the tail block when running with larger NPROMA values. Catering for too many different launch configurations in the runner script becomes unnecessarily complicated, since known good configurations are already encoded as ctests in the CMake configuration. Therefore, I've switched the CI runs to use CTest now (but kept the target validation in place).