Repository navigation
Workflows: add more eamxx-v1 tests on ghci-snl runners #8564
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 all commits
63d1992
8191ba7
b96c75f
796696a
0119149
e4e60d0
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 |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| string (APPEND KOKKOS_OPTIONS " -DKokkos_ENABLE_CUDA=ON -DKokkos_ARCH_HOPPER90=ON -DKokkos_ENABLE_CUDA_LAMBDA=ON") | ||
|
|
||
| # Change to ON if/when our runners feature cuda-aware MPI | ||
| set(SCREAM_MPI_ON_DEVICE OFF CACHE STRING "") |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| string (APPEND KOKKOS_OPTIONS " -DKokkos_ENABLE_OPENMP=ON -DKokkos_ARCH_SPR=ON") | ||
|
|
||
| # Help find_package to find the correct BLAS (i.e., the MKL installations) | ||
| set (BLA_VENDOR "Intel10_64lp" CACHE STRING "") |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2148,8 +2148,8 @@ | |
| <NODENAME_REGEX>^[a-fA-F0-9]{12}$</NODENAME_REGEX> | ||
| <OS>LINUX</OS> | ||
| <PROXY>proxy.sandia.gov:80</PROXY> | ||
| <COMPILERS>gnu</COMPILERS> | ||
| <MPILIBS>openmpi</MPILIBS> | ||
| <COMPILERS>gnu,intel,gnugpu</COMPILERS> | ||
| <MPILIBS>openmpi,mpich</MPILIBS> | ||
| <CIME_OUTPUT_ROOT>/tmp</CIME_OUTPUT_ROOT> | ||
| <DIN_LOC_ROOT>/projects/e3sm/data/inputdata</DIN_LOC_ROOT> | ||
| <DIN_LOC_ROOT_CLMFORC>/projects/e3sm/data/inputdata/atm/datm7</DIN_LOC_ROOT_CLMFORC> | ||
|
|
@@ -2162,6 +2162,7 @@ | |
| <SUPPORTED_BY>lbertag at sandia dot gov</SUPPORTED_BY> | ||
| <MAX_TASKS_PER_NODE>32</MAX_TASKS_PER_NODE> | ||
| <MAX_MPITASKS_PER_NODE>32</MAX_MPITASKS_PER_NODE> | ||
| <MAX_MPITASKS_PER_NODE compiler="gnugpu">2</MAX_MPITASKS_PER_NODE> | ||
|
Member
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. why was this PR messing with aurora?
Contributor
Author
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. This is the ghci-snl entry of config_machines though. There's one mod to aurora.cmake (not sure why), but only in comments, which should not affect anything. |
||
| <mpirun mpilib="default"> | ||
| <executable>mpirun</executable> | ||
| <arguments> | ||
|
|
||
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.
This changed answers for EAM cases on chrysalis.
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.
I had to add this, as
-fp-model sourceis no longer a valid intel flag. @amametjanov has a similar mod on his PR to fix aurora.That said, I don't think it explains anything. The file
<compiler>.cmakeis included by cmake before<machine_compiler>.cmake. Andchrysalis_intel.cmakealready hasSo this change ONLY affects builds that did NOT set the fp model in
<machine_compiler>.cmake. And if they didn't, oneapi should have errored out. Like it did on Aurora (hence, Az's PR).