Add HPC tests for the combined internal apps #506
+447
−186
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
|
Job VTB Documentation, step Sync Docs on 8f6fe6b wanted to post the following: View the site here This comment will be updated on new commits. |
|
Looks like we need to figure out how to request more procs than needed for memory |
loganharbour
force-pushed
the
PR_hpc_others
branch
2 times, most recently
from
95f0aef to
9b04aae
Compare
|
it would for sure help. I might be able to do without for now. Possibly by transitioning to a distributed mesh |
GiudGiud
force-pushed
the
PR_hpc_others
branch
from
9b04aae to
48d96c5
Compare
|
@loganharbour what's the fix when I get failures like this? |
…ibuted mesh to lower memory use
- use superlu-dist as mumps is broken rn - segregate heat transfer systems away from pressure-velocity for performance - use less transfers with the vector variables input
- remove deprecation - use general field transfers to enable a distributed mesh
- remove deprecation - add postprocessing on initial - use the intiial mesh for BISON to avoid a restart error (to be fixed) - use the restart mesh for Griffin as it's a restart simulation - remove initial conditions, not right for a restart simulation
- disable RIA HPC for now, until we fix other issues - request enough resources to avoid 4 hour runs in steady state - wiggle mesh choices to avoid restart mesh error - limit tolerances to what is achieved - only solve 3 steps for transient
…MR assembly calcs
Further increase timeout limit on KRUSTY simulation
…x checks with a solve Make sure to create a checkpoint in the GCMR steady case to be able to restart it in the transient Re-gold BISON gcmr case due to execution of PPs on initial (nicer)
GiudGiud
force-pushed
the
PR_hpc_others
branch
from
8656f8a to
c910c89
Compare
GiudGiud
force-pushed
the
PR_hpc_others
branch
2 times, most recently
from
269f528 to
ceb8c3d
Compare
GiudGiud
force-pushed
the
PR_hpc_others
branch
from
ceb8c3d to
8f6fe6b
Compare
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.
No description provided.