Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix rt.sh for 35 day tests #157

Merged
merged 2 commits into from
Aug 7, 2020
Merged

fix rt.sh for 35 day tests #157

merged 2 commits into from
Aug 7, 2020

Conversation

MinsukJi-NOAA
Copy link
Collaborator

This PR address issue #156

When running 35 day tests, the recommended command is rt.sh -l rt_35d_*.conf -cek:

  • RTPWD does not have to be changed to 20200530
  • Results will be in RUNDIR

@DeniseWorthen
Copy link
Collaborator

This appears to be working correctly for my CICE6 branch. The 20120101 cold start file was copied into the REGRESSION_TEST directory called MEDIATOR_bm_cmeps_2012010100; it was copied from there into the 35d run directory.

@MinsukJi-NOAA
Copy link
Collaborator Author

This appears to be working correctly for my CICE6 branch. The 20120101 cold start file was copied into the REGRESSION_TEST directory called MEDIATOR_bm_cmeps_2012010100; it was copied from there into the 35d run directory.

That is correct. Although -c flag means create a new baseline (i.e. REGRESSION_TEST directory), this is just a dummy (or intermediate) location to transfer new mediator files from the cold runs to warm runs. For test results, you should go back to the run directory (and not REGRESSION_TEST directory); this is why you also need -k flag to not delete the RUNDIR.

@MinsukJi-NOAA MinsukJi-NOAA marked this pull request as ready for review August 7, 2020 18:21
@DeniseWorthen DeniseWorthen merged commit 10b2e46 into ufs-community:develop Aug 7, 2020
@MinsukJi-NOAA MinsukJi-NOAA deleted the fix/rt-35d branch August 7, 2020 20:22
@MinsukJi-NOAA MinsukJi-NOAA mentioned this pull request Aug 7, 2020
Closed
binli2337 pushed a commit that referenced this pull request Sep 28, 2020
@climbfuji
Update .gitmodules and submodule pointer for fv3atm for PR #157 (RRTM…
fab7168 
…G cloud overlap method update) (#193)
MinsukJi-NOAA pushed a commit to MinsukJi-NOAA/ufs-s2s-model that referenced this pull request Oct 7, 2020
@BrianCurtis-NOAA @binli2337
@climbfuji @DeniseWorthen @aerorahul
Rt unified ufs bc (#4)
4665549 
* Use hpc-stack libraries in ufs-s2s-model (ufs-community#191)

* Update .gitmodules and submodule pointer for fv3atm for PR ufs-community#157 (RRTMG cloud overlap method update) (ufs-community#193)

* Bugfix/wave1dtest (ufs-community#195)

* fix MOM6 input template used by the cpld_fv3_ccpp_384_mom6_cice_cmeps_ww3_1d_bmark_rt test

* Remove PIO as part of S2S model build in CMEPS and use hpc-stack (ufs-community#199)

* update CMEPS submodule pointer (ufs-community#205)

* Merge with upstream and fix conflicts

Co-authored-by: BinLi-NOAA <bin.li@noaa.gov>
Co-authored-by: Dom Heinzeller <dom.heinzeller@icloud.com>
Co-authored-by: Denise Worthen <denise.worthen@noaa.gov>
Co-authored-by: Rahul Mahajan <aerorahul@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BrianCurtis-NOAA BrianCurtis-NOAA BrianCurtis-NOAA left review comments

@junwang-noaa junwang-noaa junwang-noaa approved these changes

@DeniseWorthen DeniseWorthen DeniseWorthen approved these changes

@binli2337 binli2337 Awaiting requested review from binli2337

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MinsukJi-NOAA @DeniseWorthen @junwang-noaa @BrianCurtis-NOAA