-
Notifications
You must be signed in to change notification settings - Fork 110
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 9-deg ocean+ice file generation, specification of ATM resolutions via namelist #1013
base: develop
Are you sure you want to change the base?
add 9-deg ocean+ice file generation, specification of ATM resolutions via namelist #1013
Conversation
* link to a temp directory w/ additional ATM grids * add version number (date) to MOM6 fix in rt.sh * add 9-deg resolution to post
* baseline successfully created on hera for all ocn/atm combinations
@GeorgeGayno-NOAA I've been able to use the fix file location on Hera to generate all the mapped ocean masks for C12 thru C3072. Generation of the C3072 extends the time of the RT to more than an hour though. This is because the RegridWeight call happens in serial (even if the code were modified to run on more than 1 PET). One note: All the mapped ocean masks currently in the fix directory (ie, |
To reduce the RT wall clock time, do we need to test every combination? Can we just do a subset? If I set up the test as follows:
Everything runs in about 24 minutes:
|
Another way to speed things up? Could the mx??? tests be run in parallel? Most UFS_UTILS tests run that way. For example, see the |
@GeorgeGayno-NOA Regarding running in parallel, I'm still relying on the intial rt.sh functionality that Minsuk put in. I don't really have the time to better develop a RT testing framework for cpld-gridgen. Do you have someone on your side who could take that on? In terms of what to generate, there are two really issues I think. One, is a generating a sufficient baseline to test PRs against---that doesn't have to be every combination. A second is the production of "things" needed to test various workflows (like the post-weights). Right now, we're populating the fix files w/ the results of the baseline RTs, which produces the cpld fix files required for any combination up to C1152. That gives workflows a simple place to grab anything they might need (eg post weights for C192mx025). I don't know if that is the right system though. Should GW be asked to generate what they need themselves? The simplest solution is to leave C3072 off and maintain what we currently have--the RTs would run in the same time
|
I was just curious it that is feasible. Maybe I can find time to work on it.
I have no problem asking people to generate what they need. i.e., the rt.conf file would be setup for the regression testing and users can adjust it for their particular project. But let's just keep what we have for now. Can you baseline what you have above? |
@GeorgeGayno-NOAA Sure, I will create baselines. Are the new C12 and C24 fix available everywhere (they weren't on hercules when I looked earlier today). |
DESCRIPTION OF CHANGES:
One or more paragraphs describing the problem, solution, and required changes.
TESTS CONDUCTED:
If there are changes to the build or source code, the tests below must be conducted. Contact a repository manager if you need assistance.
cpld_gridgen
test locally on all Tier 1 machines.Describe any additional tests performed.
DEPENDENCIES:
Add any links to pending PRs that are required prior to merging this PR. For example:
ufs-community/UFS_UTILS/pull/<pr_number>
DOCUMENTATION:
If this PR is contributing new capabilities that need to be documented, please also include updates to the RST files in the docs/source directory as supporting material.
ISSUE:
Fixes #1004
CONTRIBUTORS (optional):
If others have contributed to this work aside from the PR author, list them here