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.

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 ability to use GEFS replay ICs #2559

Merged
merged 100 commits into from
Jun 13, 2024

Conversation

NeilBarton-NOAA
Copy link
Contributor

Description

The PR allows the use of ICs from PSL's replay analysis. These replay ICs will be used for GEFS reforecasting and SFS. Two main changes are associated with these updates: (1) replay ICs being valid at 3Z, and (2) the use of warm starts.

This addresses #1838

Type of change

  • New feature

Change characteristics

  • Is this a breaking change (a change in existing functionality)? NO
  • Does this change require a documentation update? NO

How has this been tested?

  • GEFS Forecast-only on Hera using replay ICs
  • GEFS Forecast-only on Hera using warm start ICs
  • GEFS Forecast-only on Hera using cold start ICs
  • GFS Forecast-only on Hera CI test

Checklist

  • Any dependent changes have been merged and published
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • New and existing tests pass with my changes
  • I have made corresponding changes to the documentation if necessary

@@ -247,8 +247,9 @@ source "${USHgfs}/preamble.sh"
-e "s/FORMAT/F/g" \
ww3_outp_spec.inp.tmpl > ww3_outp.inp

${NLN} mod_def.$waveuoutpGRD mod_def.ww3
HMS="${cyc}0000"
ln -s mod_def.$waveuoutpGRD mod_def.ww3
Copy link
Contributor

Choose a reason for hiding this comment

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

Please retain the use of ${NLN}

scripts/exglobal_stage_ic.sh Outdated Show resolved Hide resolved
@@ -149,7 +171,7 @@ for MEMDIR in "${MEMDIR_ARRAY[@]}"; do

# Stage the WW3 initial conditions to ROTDIR (warm start; TODO: these should be placed in $RUN.$gPDY/$gcyc)
if [[ "${DO_WAVE:-}" = "YES" ]]; then
YMD=${PDY} HH=${cyc} declare_from_tmpl COM_WAVE_RESTART
YMD=${gPDY} HH=${gcyc} declare_from_tmpl COM_WAVE_RESTART
Copy link
Contributor

Choose a reason for hiding this comment

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

If this is now going to use the previous cycle's date, the variable name should be changed to COM_WAVE_RESTART_PREV to be consistent.

Also, I believe this will break the existing GEFS test.

# Atmosphere Perturbation Files (usually used with replay ICS)
# Extra zero on MEMDIR ensure we have a number even if the string is empty
if (( 0${MEMDIR:3} > 0 )) && [[ "${USE_ATM_PERTURB_FILES:-false}" == "true" ]]; then
src="${BASE_CPLIC}/${CPL_OCNIC:-}/${PDY}${cyc}/${MEMDIR}/atmos/${PDY}.${cyc}0000.fv3_perturbation.nc"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are atmosphere perturbations coming from the ocean IC directory? Doesn't matter right now since they are identical, but this should use CPL_ATMIC.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Error in coping and pasting. Updated

@@ -97,16 +100,28 @@ FV3_postdet() {
local model_start_time="${current_cycle}"
local model_current_time="${current_cycle}"
fi
if [[ "${USE_REPLAY_ICS}" != "true" ]]; then
rm -f "${DATA}/INPUT/coupler.res"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please indent this block (except the EOF)

@@ -490,6 +505,7 @@ MOM6_out() {
esac

# Copy MOM6 restarts at the end of the forecast segment to COM for RUN=gfs|gefs
if [[ "${USE_REPLAY_ICS}" != "true" ]]; then
local restart_file
Copy link
Contributor

Choose a reason for hiding this comment

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

Please indent this block.

@@ -590,6 +607,7 @@ CICE_out() {
${NCP} "${DATA}/ice_in" "${COM_CONF}/ufs.ice_in"

# Copy CICE restarts at the end of the forecast segment to COM for RUN=gfs|gefs
if [[ "${USE_REPLAY_ICS}" != "true" ]]; then
local seconds source_file target_file
Copy link
Contributor

Choose a reason for hiding this comment

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

Please indent this block.

@@ -729,6 +748,7 @@ CMEPS_out() {
echo "SUB ${FUNCNAME[0]}: Copying output data for CMEPS mediator"

# Copy mediator restarts at the end of the forecast segment to COM for RUN=gfs|gefs
if [[ "${USE_REPLAY_ICS}" != "true" ]]; then
echo "Copying mediator restarts for 'RUN=${RUN}' at ${forecast_end_cycle}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please indent this block.

@WalterKolczynski-NOAA WalterKolczynski-NOAA changed the title Replay ics Add ability to use GEFS replay ICs May 1, 2024
@WalterKolczynski-NOAA
Copy link
Contributor

Does this resolve #1838 or is it only part of it? The details were never filled in to that issue to know what needs to be done.

@emcbot emcbot added CI-Hera-Running **Bot use only** CI testing on Hera for this PR is in-progress CI-Hera-Passed **Bot use only** CI testing on Hera for this PR has completed successfully and removed CI-Hera-Ready **CM use only** PR is ready for CI testing on Hera CI-Hera-Building **Bot use only** CI testing is cloning/building on Hera CI-Hera-Running **Bot use only** CI testing on Hera for this PR is in-progress labels Jun 12, 2024
@emcbot
Copy link

emcbot commented Jun 12, 2024

CI Passed Hera at
Built and ran in directory /scratch1/NCEPDEV/global/CI/2559

@WalterKolczynski-NOAA WalterKolczynski-NOAA added CI-Hercules-Ready **CM use only** PR is ready for CI testing on Hercules CI-Wcoss2-Ready **CM use only** PR is ready for CI testing on WCOSS labels Jun 12, 2024
@emcbot emcbot added CI-Wcoss2-Building **Bot use only** CI testing is cloning/building on WCOSS and removed CI-Wcoss2-Ready **CM use only** PR is ready for CI testing on WCOSS labels Jun 12, 2024
@emcbot
Copy link

emcbot commented Jun 12, 2024

CI Update on Wcoss2 at 06/12/24 09:40:18 PM
============================================
Cloning and Building global-workflow PR: 2559
with PID: 181360 on host: clogin05

@emcbot emcbot added CI-Hercules-Building **Bot use only** CI testing is cloning/building on Hercules CI-Hercules-Running **Bot use only** CI testing on Hercules for this PR is in-progress CI-Wcoss2-Running **Bot use only** CI testing on WCOSS for this PR is in-progress and removed CI-Hercules-Ready **CM use only** PR is ready for CI testing on Hercules CI-Hercules-Building **Bot use only** CI testing is cloning/building on Hercules CI-Wcoss2-Building **Bot use only** CI testing is cloning/building on WCOSS labels Jun 12, 2024
@emcbot
Copy link

emcbot commented Jun 12, 2024

Automated global-workflow Testing Results:

Machine: Wcoss2
Start: Wed Jun 12 21:57:56 UTC 2024 on clogin05
---------------------------------------------------
Build: Completed at 06/12/24 10:53:28 PM
Case setup: Completed for experiment C48_ATM_7a383978
Case setup: Skipped for experiment C48mx500_3DVarAOWCDA_7a383978
Case setup: Skipped for experiment C48_S2SWA_gefs_7a383978
Case setup: Completed for experiment C48_S2SW_7a383978
Case setup: Completed for experiment C96_atm3DVar_extended_7a383978
Case setup: Skipped for experiment C96_atm3DVar_7a383978
Case setup: Skipped for experiment C96_atmaerosnowDA_7a383978
Case setup: Completed for experiment C96C48_hybatmDA_7a383978
Case setup: Completed for experiment C96C48_ufs_hybatmDA_7a383978

@emcbot
Copy link

emcbot commented Jun 13, 2024

Experiment C48_ATM_7a383978 SUCCESS on Wcoss2 at 06/13/24 12:36:15 AM

@emcbot
Copy link

emcbot commented Jun 13, 2024

Experiment C48_S2SW_7a383978 SUCCESS on Wcoss2 at 06/13/24 01:06:10 AM

@emcbot
Copy link

emcbot commented Jun 13, 2024

Experiment C96C48_hybatmDA_7a383978 SUCCESS on Wcoss2 at 06/13/24 01:36:18 AM

@emcbot
Copy link

emcbot commented Jun 13, 2024

Experiment C96C48_ufs_hybatmDA_7a383978 SUCCESS on Wcoss2 at 06/13/24 01:48:17 AM

@emcbot
Copy link

emcbot commented Jun 13, 2024

Experiment C96_atm3DVar_extended_7a383978 SUCCESS on Wcoss2 at 06/13/24 07:27:30 AM

@emcbot emcbot added CI-Wcoss2-Passed **Bot use only** CI testing on WCOSS for this PR has completed successfully and removed CI-Wcoss2-Running **Bot use only** CI testing on WCOSS for this PR is in-progress labels Jun 13, 2024
@emcbot
Copy link

emcbot commented Jun 13, 2024

All CI Test Cases Passed on Wcoss2:

Experiment C48_ATM_7a383978 *** SUCCESS *** at 06/13/24 12:36:15 AM
Experiment C48_S2SW_7a383978 *** SUCCESS *** at 06/13/24 01:06:10 AM
Experiment C96C48_hybatmDA_7a383978 *** SUCCESS *** at 06/13/24 01:36:18 AM
Experiment C96C48_ufs_hybatmDA_7a383978 *** SUCCESS *** at 06/13/24 01:48:17 AM
Experiment C96_atm3DVar_extended_7a383978 *** SUCCESS *** at 06/13/24 07:27:30 AM

@TerrenceMcGuinness-NOAA TerrenceMcGuinness-NOAA removed the CI-Hercules-Running **Bot use only** CI testing on Hercules for this PR is in-progress label Jun 13, 2024
@WalterKolczynski-NOAA WalterKolczynski-NOAA merged commit 34155fb into NOAA-EMC:develop Jun 13, 2024
7 of 10 checks passed
danholdaway added a commit to danholdaway/global-workflow that referenced this pull request Jun 13, 2024
* upstream/develop:
  Add Hercules-EMC to the Jenkins configurable parameter list (NOAA-EMC#2685)
  Update gdas.cd and gsi_utils hashes (NOAA-EMC#2641)
  Add ability to use GEFS replay ICs (NOAA-EMC#2559)
  Replace `sleep` with `wait_for_file` (NOAA-EMC#2586)
KateFriedman-NOAA added a commit to KateFriedman-NOAA/global-workflow that referenced this pull request Jun 14, 2024
* origin/develop:
  Add observation preparation job for aerosols DA to workflow (NOAA-EMC#2624)
  Remove ocean daily files (NOAA-EMC#2689)
  Update Jenkinsfile
  Add Hercules-EMC to the Jenkins configurable parameter list (NOAA-EMC#2685)
  Update gdas.cd and gsi_utils hashes (NOAA-EMC#2641)
  Add ability to use GEFS replay ICs (NOAA-EMC#2559)
  Replace `sleep` with `wait_for_file` (NOAA-EMC#2586)
  Add COM template for JEDI obs (NOAA-EMC#2678)
  Link both global-nest fix files and non-nest ones at the same time (NOAA-EMC#2632)
  Update ufs-weather-model  (NOAA-EMC#2663)
  Add ability to process ocean/ice products specific to GEFS (NOAA-EMC#2561)
  Update cleanup job to use COMIN/COMOUT (NOAA-EMC#2649)
  Add overwrite to creat experiment in BASH CI (NOAA-EMC#2676)
  Add handling to select CRTM cloud optical table based on cloud scheme and update calcanal_gfs.py  (NOAA-EMC#2645)

Refs NOAA-EMC#2475
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-Hera-Passed **Bot use only** CI testing on Hera for this PR has completed successfully CI-Wcoss2-Passed **Bot use only** CI testing on WCOSS for this PR has completed successfully
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants