-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add a restart test to cosine bell #251
Conversation
The restart test case nearly works but needs a fix for E3SM-Project/Omega#181. |
It works!!! Using a test merge of E3SM-Project/Omega#184 with the current Omega |
component=component, name=name, subdir=subdir, | ||
mesh=base_mesh_step, init=init_step, | ||
refinement_factor=refinement_factor, | ||
refinement=refinement) |
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.
refinement=refinement) | |
refinement=refinement, | |
do_restart=name == 'restart_run') |
Since you end up using the logical do_restart
in dynamic_model_config
, should we just use this as an argument rather than checking step names?
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 think this looks great! I ran successfully with MPAS-O and Omega on chrys, intel. I trust you to do the items on the checklist
This is needed if we are translating dimension and variable names between native model names and common Polaris names.
0cb3690
to
ee6d56d
Compare
Add a `do_restart` attribute to keep track of which runs are restart runs, rather than using the step name.
TestingI successfully ran both the |
I need to take care of documentation and adding tests to suites before I merge. |
Update cosine_bell suite to include all tests. Remove cosine_bell_cached_init suite, which has not been well maintained.
Okay, docs and test suites are up-to-date. |
2ac8c51
to
2b6571d
Compare
This merge adds a restart test to the cosine bell configuration. This requires some changes to the validation framework for comparing variables to support the translation that Polaris performs between Omega native dimension and variable names and those for MPAS-Ocean that the Polaris ocean component uses generally.
A small amount of related cleanup is also included.
Checklist
api.md
) has any new or modified class, method and/or functions listedTesting
comment in the PR documents testing used to verify the changes