-
Notifications
You must be signed in to change notification settings - Fork 248
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
Thompson subcycling for develop, add missing hera.gnu debug modulefile #632
Thompson subcycling for develop, add missing hera.gnu debug modulefile #632
Conversation
…h when checking for DEBUG=Y
…r-model into thompson_subcycling_v2_tutorial
@BrianCurtis-NOAA this is because you stopped them because of the GitHub token change, correct? |
I haven't stopped anything yet. and haven't switched accesstoken on Cheyenne either. |
Machine: cheyenne |
Note sure how helpful this is:
|
Machine: gaea |
The exact same tests failed again on Cray. On Dell, these failed (all overlap with Cray failed jobs): |
Machine: orion |
Machine: cheyenne |
Machine: jet |
I made a stupid mistake in the last-minute code changes based on the code review. It's fixed now, regression tests have been kicked off manually on all systems. I also started the CI tests just now. |
Dom, just to confirm, you are also running RT on wcoss, please let me know
if you have any issue, I can help to run RT on wcoss.
…On Wed, Jun 9, 2021 at 10:50 PM Dom Heinzeller ***@***.***> wrote:
I made a stupid mistake in the last-minute code changes based on the code
review. It's fixed now, regression tests have been kicked off manually on
all systems. I also started the CI tests just now.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#632 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TMGUDD7WPHPZSUEGFLTSAR67ANCNFSM46MYWL7A>
.
|
Thanks a lot. Seems to be working. I created the baseline on cray, will start verification after copying it over. Dell is still busy creating baselines. The biggest bottleneck is hera. If all other platforms finish except hera, we may have to do the merge w/o waiting for hera to finish ... |
OK, let's see what we have by tomorrow morning.
…On Wed, Jun 9, 2021 at 11:11 PM Dom Heinzeller ***@***.***> wrote:
Dom, just to confirm, you are also running RT on wcoss, please let me know
if you have any issue, I can help to run RT on wcoss.
… <#m_2346224976923207178_>
Thanks a lot. Seems to be working. I created the baseline on cray, will
start verification after copying it over. Dell is still busy creating
baselines. The biggest bottleneck is hera. If all other platforms finish
except hera, we may have to do the merge w/o waiting for hera to finish ...
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#632 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TPBYV4WWSZ7NACIU4DTSAUOJANCNFSM46MYWL7A>
.
|
CI tests passed for commit a3910a9. |
Regression tests passed on all machines. I pushed the gaea.intel log directly from the machine, all others I copied to my laptop, but didn't commit them yet. This way I can show users how to pull in the updates to simulate what they need to do when auto-rt pushes some of the log files. |
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.
Thanks for adding the hera_gnu debug modules.
Once all the RT log files are committed, the code can be committed.
…632) * Build UPP for AQM (Online-CMAQ). * Add four new cycles to the workflow XML file for real-time run with varying forecast length hours. * Update the UPP input namelist and control file. --------- Co-authored-by: chan-hoo <chan-hoo.jeon@clogin04.cactus.wcoss2.ncep.noaa.gov>
PR Checklist
Ths PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR. Please consult the ufs-weather-model wiki if you are unsure how to do this.
This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR
An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
are specified below.
If new or updated input data is required by this PR, it is clearly stated in the text of the PR.
Description
Implement a subcycling capability for Thompson MP in CCPP and exercise it in the UFS regression tests. This ufs-weather-model PR only updates the submodule pointer for fv3atm for the changes w.r.t. Thompson MP described in the associated PRs below.
Additional changes:
ufs_hera.gnu_debug
compile.sh
to copy the correct modulefile when DEBUG is usedThe changes to the suite FV3_GSD_noah/FV3_GFS_v16_thompspn (run Thompson MP with 4/2 subcycles) change the answer of several Thompson MP based regression tests. No new input data required.
Issue(s) addressed
Fixes #596
Testing
Preliminary regression testing on Hera with GNU and Intel against existing baseline (2021/05/26): all tests that are expected to pass do pass, and all tests that are expected to fail do fail with b4b mismatches (but they all run to completion):
rt_hera_gnu_verify_against_existing.log
rt_hera_gnu_verify_against_existing_fail_test.log
rt_hera_initel_verify_against_existing.log
rt_hera_initel_verify_against_existing_fail_test.log
Full regression tests will be run on all tier-1 platforms when it is time to commit.
Dependencies
NCAR/ccpp-framework#379
NCAR/ccpp-physics#676
NOAA-EMC/fv3atm#328
#632