-
Notifications
You must be signed in to change notification settings - Fork 171
add nvhpc to test suite github runners #1317
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
Changes from 56 commits
3ebc4a7
790dfbd
e72b8fc
4f44702
fed16a2
00857fe
f38b5a2
581645b
5bfaed3
23a0bd9
5ef74e3
5e8b22b
65bc2f8
066089f
9472f4d
ff30d8a
dc12849
627d01a
4ad88a9
20288ec
f64894f
1a60cbe
419132c
b30a38b
2c4dd4b
ebf82b4
26d82be
fb2b897
f38948b
d47b0d7
e623a0e
35f8160
ef07069
1d5249d
2e6a5ff
cf4e6c6
8132d1e
36d5020
809c4c6
f99fadc
f9ddc91
d1fa235
1cff5eb
206bfe7
cb4c3ea
1cb6fec
e8847dc
63ece30
fc24f44
a1ff384
b6e430e
61f2500
0c6bbde
88133d3
98ca704
3523461
9b97a59
4b3fed4
a322518
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -187,6 +187,15 @@ def __filter(cases_) -> typing.Tuple[typing.List[TestCase], typing.List[TestCase | |
| if any(label in case.trace for label in skip): | ||
| cases.remove(case) | ||
|
|
||
| # Skip Lagrange Bubbles tests under nvfortran in Docker: NaN on 24.1/24.3 | ||
| # due to unresolved Docker-specific numerical issue (passes natively/Apptainer). | ||
| if os.environ.get("FC") == "nvfortran" and os.path.exists("/.dockerenv"): | ||
| nvhpc_skip_uuids = {"B9553426", "4A1BD9B8", "0D1FA5C5", "2122A4F6"} | ||
| for case in cases[:]: | ||
| if case.get_uuid() in nvhpc_skip_uuids: | ||
| cases.remove(case) | ||
| skipped_cases.append(case) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: # Check the workflow file for NVHPC versions
cat .github/workflows/test.yml | head -100Repository: MFlowCode/MFC Length of output: 3414 🏁 Script executed: # Find and examine the test.py file around the skip logic
sed -n '180,210p' toolchain/mfc/test/test.pyRepository: MFlowCode/MFC Length of output: 1626 🏁 Script executed: # Check if there's any environment variable or method that carries NVHPC version
grep -n "NVHPC\|nvhpc\|nvfortran" toolchain/mfc/test/test.py | head -20Repository: MFlowCode/MFC Length of output: 356 🏁 Script executed: # Find NVHPC or nvhpc references and matrix configuration in the workflow
grep -n -i "nvhpc\|matrix" .github/workflows/test.yml | head -30Repository: MFlowCode/MFC Length of output: 1624 🏁 Script executed: # Get a larger section of the workflow file to see the test job configuration
sed -n '100,300p' .github/workflows/test.ymlRepository: MFlowCode/MFC Length of output: 8086 🏁 Script executed: # Check if NVHPC version is set as an environment variable in test.py or workflow
grep -n "NVHPC_VERSION\|nvhpc.*version\|version.*nvhpc" toolchain/mfc/test/test.py .github/workflows/test.yml 2>/dev/nullRepository: MFlowCode/MFC Length of output: 39 Make this NVHPC skip version-aware. The comment documents the failure as 24.1/24.3, but the predicate only checks |
||
|
|
||
| if ARG("no_examples"): | ||
| example_cases = [case for case in cases if "Example" in case.trace] | ||
| skipped_cases += example_cases | ||
|
|
||
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.
Don't globally skip a Docker-only failure.
The comment says the crash is specific to
nvfortran + MPIin Docker, but adding3D_rayleigh_taylor_musclhere removes it from every compiler/platform, including native runs and non-NVHPC CI. Please move this to the environment-specific filtering intoolchain/mfc/test/test.py(or gate it on the same Docker/NVHPC condition) so the other jobs still exercise the example.