Skip to content

[rrfs-mpas-jedi]Hotfix for continuous surface cycling#1558

Merged
MatthewPyle-NOAA merged 4 commits into
NOAA-EMC:rrfs-mpas-jedifrom
chunhuazhou:hotfix_v2X
Jun 22, 2026
Merged

[rrfs-mpas-jedi]Hotfix for continuous surface cycling#1558
MatthewPyle-NOAA merged 4 commits into
NOAA-EMC:rrfs-mpas-jedifrom
chunhuazhou:hotfix_v2X

Conversation

@chunhuazhou

Copy link
Copy Markdown
Collaborator

DESCRIPTION OF CHANGES:

Problem: Previous failed tasks cause cycling to pause in RRFSv2X.
Solution: This hotfix facilitates continuous surface cycling by:

  • Adding a 2-hour time dependency for prep_ic.
  • Outputting more mpasout files to allow the system to look back further for required cycling data.

TESTS CONDUCTED:

These changes are applied in RRFSv2X.

@guoqing-noaa

guoqing-noaa commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

To confirm, this PR would like to use the latest 1,2,3hrs forecasts for surface cycling if possible; if we can not get any of them in 3 hours, the prep_ic task will start anyway but then use a longer forecast (15 or 27h). Do I get it right?

+        <timedep><cyclestr offset="03:00:00">@Y@m@d@H@M00</cyclestr></timedep>
         <datadep age="00:00:05"><cyclestr offset="-1:00:00">&COMROOT;/&NET;/&rrfs_ver;/&RUN;.@Y@m@d/@H/fcst/&WGF;/fcst_f001.done</cyclestr></datadep>
         <datadep age="00:00:05"><cyclestr offset="-2:00:00">&COMROOT;/&NET;/&rrfs_ver;/&RUN;.@Y@m@d/@H/fcst/&WGF;/fcst_f002.done</cyclestr></datadep>
         <datadep age="00:00:05"><cyclestr offset="-3:00:00">&COMROOT;/&NET;/&rrfs_ver;/&RUN;.@Y@m@d/@H/fcst/&WGF;/fcst_f003.done</cyclestr></datadep>

@chunhuazhou

Copy link
Copy Markdown
Collaborator Author

To confirm, this PR would like to use the latest 1,2,3hrs forecasts for surface cycling if possible; if we can not get any of them in 3 hours, the prep_ic task will start anyway but then use a longer forecast (15 or 27h). Do I get it right?

+        <timedep><cyclestr offset="03:00:00">@Y@m@d@H@M00</cyclestr></timedep>
         <datadep age="00:00:05"><cyclestr offset="-1:00:00">&COMROOT;/&NET;/&rrfs_ver;/&RUN;.@Y@m@d/@H/fcst/&WGF;/fcst_f001.done</cyclestr></datadep>
         <datadep age="00:00:05"><cyclestr offset="-2:00:00">&COMROOT;/&NET;/&rrfs_ver;/&RUN;.@Y@m@d/@H/fcst/&WGF;/fcst_f002.done</cyclestr></datadep>
         <datadep age="00:00:05"><cyclestr offset="-3:00:00">&COMROOT;/&NET;/&rrfs_ver;/&RUN;.@Y@m@d/@H/fcst/&WGF;/fcst_f003.done</cyclestr></datadep>

Yes, that is right. Thanks!

@guoqing-noaa guoqing-noaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Thanks, @chunhuazhou

@MatthewPyle-NOAA MatthewPyle-NOAA merged commit a2e29ea into NOAA-EMC:rrfs-mpas-jedi Jun 22, 2026
7 checks passed
@chunhuazhou chunhuazhou deleted the hotfix_v2X branch June 22, 2026 20:48
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.

3 participants