Skip to content

Feature/rrfs dev1: add retrospective function and changes for running on WCOSS#104

Merged
hu5970 merged 16 commits into
NOAA-GSL:feature/RRFS_dev1from
hu5970:feature/RRFS_dev1
May 13, 2021
Merged

Feature/rrfs dev1: add retrospective function and changes for running on WCOSS#104
hu5970 merged 16 commits into
NOAA-GSL:feature/RRFS_dev1from
hu5970:feature/RRFS_dev1

Conversation

@hu5970
Copy link
Copy Markdown

@hu5970 hu5970 commented May 12, 2021

This PR to add the following functions to RRFS:

  1. Code update for running on WCOSS with FV3GFS background.
  2. Run retrospective experiment on WCOSS when the data are still on disk.
    This also builds the same dependences for all retrospective experiments.

hu5970 added 14 commits April 28, 2021 22:58
Update RRFS_dev1 configure file for running 48/18h forecast.
…g experiment.

RRFS dev1 CONUS:
Hourly cycled with forecasts to 18 hrs except 48 hrs at 00/06/12/18z, cold-started at 09z/21z from HRRR, boundary from RAP.

RRFS dev1 AK:
Hourly cycled with forecasts to 21 hrs, 63 hours at 03/09/15/21z, cold-started at 03z/15z from GFS

RRFS dev1 NA 13km:
3-hourly cycled with forecasts to 18 hrs, cold-started at 09z/21z from RAP, boundary from RAP
Use rap_e for 00/12Z cycle.
… one task for both long and normal forecast.
Add function to run retrospective experiment on WCOSS when the data are still on disk.
Copy link
Copy Markdown
Collaborator

@christinaholtNOAA christinaholtNOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ming, this looks pretty good, although I don't know what is needed on WCOSS.

I've made several comments that are very similar to comments I've made on previous PRs. Please keep them in mind when adding new features.


DO_DACYCLE="true"
#DO_RETRO="true"
#LBCS_ICS_ONLY="true"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove commented configurations.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will use the retro functions a a lot soon. This is not just a comment. This is a place hold for retro because ti is much easier and safe to uncomment those options for retrospective runs.

;;
"JET")

for loop in $loops; do
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use the $() syntax instead of backticks for clarity, ease of debugging, and supportability here and in the previous case statement.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, change all in this scripts.

DATE_FIRST_CYCL="20210401"
DATE_LAST_CYCL="20211231"
CYCL_HRS=( "00" "12" )
DT_ATMOS=30
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this an intended change for RRFS_dev1 in real time?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that is the number in real-time runs now.

Comment thread ush/config.sh.RRFS_dev1 Outdated

DO_DACYCLE="true"
DO_RETRO="false"
LBCS_ICS_ONLY="false"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be the default, and shouldn't need to be set here, right?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, I would like to leave them as a comment to make retro run easier. many colleagues may not know those options.

Comment thread ush/config.sh.RRFS_dev1
POSTPROC_CYCLEDEF="00 00-23/01 ${CYCLEDAY} ${CYCLEMONTH} 2021 *"
POSTPROC_LONG_CYCLEDEF="00 00,06,12,18 ${CYCLEDAY} ${CYCLEMONTH} 2021 *"
ARCHIVE_CYCLEDEF="00 07 ${CYCLEDAY} ${CYCLEMONTH} 2021 *"
#CYCLEDAY="1-4"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How and where is CYCLEDAY set? If 1-4 is a default value, please remove from the config file for clarity.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The default value is "*" but I use this one always in my test. It is much easier to leave it as place hold for testing.
Our code is under active development, there are some commend lines are there for easy pick up during develop and test.

Comment thread ush/config.sh.RRFS_wcoss
COMINgfs=""

STMP="/gpfs/hps3/ptmp/Ming.Hu/stmp" # Path to directory STMP that mostly contains input files.
PTMP="/gpfs/hps3/ptmp/Ming.Hu/ptmp" # Path to directory STMP that mostly contains input files.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please leave out personal output locations for retro runs.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are no official space on WCOSS. All tests (even parallel for operation updates) are done in person space.

else
fns_on_disk=( "gfs.t${hh}z.pgrb2.0p25.f${fcst_hh}" ) # Get only 0.25 degree files for now.
# fns_on_disk=( "gfs.t${hh}z.pgrb2.0p25.f000") # Get only 0.25 degree files for now.
fns_on_disk=( "gfs.t${hh}z.pgrb2.0p25.f0${fcst_hh}" "gfs.t${hh}z.sfcf0${fcst_hh}.nc") # use netcdf
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove commented code.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

@hu5970 hu5970 merged commit fe83181 into NOAA-GSL:feature/RRFS_dev1 May 13, 2021
hu5970 added a commit that referenced this pull request May 29, 2021
* Feature/rrfs dev1: add retrospective function and changes for running on WCOSS (#104)

* Cold start RRFS_dev1 3km CONUS hourly cycles from 09/21Z HRRR analysis.
* Changes for running on RRFS cycles on WCOSS.

* Copy addtional grid from UPP run directory to comout space. (#105)

* Copy addtional grid from UPP run directory to comout space.

* Improvements for 3-km NA (#103)

* Extending walltime for "run_post" in 3-km NA

* Updated settings for layout_x, layout_y to improve run time.

* Increase RRFS_AK time step to match CONUS (#110)

* Feature/rrfs dev1: add retro and radar tten functions (#108)

* Add functions to run retro with staged data on disk.
* Add namelist option for radar tten.
* Bug fix for using GFS on JET for 3km NA.

* Feature/rrfs dev1: delete radar tten option (#112)

* delete tten option in namelist to match current USD model.

* Turn off cloud analysis for future debug. (#113)

* Change the "overwrite" option to "upgrade" (#111)

* Change $EXPDIR backup from inside to outside (#117)

* RRFS_dev1: Add python maps (#82)

* Updating job script to work with maps.

* Workflow for Python graphics.

* Use a metatask to manage map subsets.

Move skewt out of metatask.

* Sorting out plotting the wgrib2-produced domains.

* Updating configure scripts for appropriate domains.

* Adding domains for 3DRTMA

* Fixes for undefined variables and logic changes.

* RRFS_dev1: Turn on radar tten option and use "upgrade" to deploy. (#121)

* Add and turn on radar tten option
* Change PREEXISTING_DIR_METHOD from "rename" to "upgrade" for all RRFS_dev runs.

* Add configure options to decide how to clean the old cycles. (#123)

Change the reference date for clean from current run time to current cycle time

Co-authored-by: Jaymes Kenyon <jaymes.kenyon@noaa.gov>
Co-authored-by: Trevor Alcott <69324222+TrevorAlcott-NOAA@users.noreply.github.com>
Co-authored-by: Guoqing Ge <guoqing.ge@noaa.gov>
Co-authored-by: Christina Holt <56881914+christinaholtNOAA@users.noreply.github.com>
EdwardColon-NOAA pushed a commit to EdwardColon-NOAA/regional_workflow that referenced this pull request Sep 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants