Skip to content

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

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

Merged
hu5970 merged 2 commits into from
May 24, 2021
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
55974eb
Turn on rdaar tten option and use "upgrade" to deploy.
hu5970 May 24, 2021
e90c090
Change PREEXISTING_DIR_METHOD from "rename" to "upgrade" for all RRFS…
hu5970 May 24, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion scripts/exregional_refl2tten.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ else # cycle uses background from
ln_vrfy -s ${bkpath}/fv_core.res.tile1.nc fv3_dynvars
ln_vrfy -s ${bkpath}/fv_tracer.res.tile1.nc fv3_tracer
ln_vrfy -s ${bkpath}/sfc_data.nc fv3_sfcdata
cp_vrfy ${bkpath}/phy_data.nc fv3_phydata
ln_vrfy -s ${bkpath}/phy_data.nc fv3_phydata
fi

#
Expand Down
2 changes: 1 addition & 1 deletion ush/config.sh.RRFS_AK_dev1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ FIX_CRTM=/home/rtrr/FIX_EXEC_MODULE/crtm/CRTM_v2.3.0
VERBOSE="TRUE"

RUN_ENVIR="nco"
PREEXISTING_DIR_METHOD="rename"
PREEXISTING_DIR_METHOD="upgrade"

PREDEF_GRID_NAME=RRFS_AK_3km

Expand Down
2 changes: 1 addition & 1 deletion ush/config.sh.RRFS_NA_13km
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ FIX_CRTM=/home/rtrr/FIX_EXEC_MODULE/crtm/CRTM_v2.3.0
VERBOSE="TRUE"

RUN_ENVIR="nco"
PREEXISTING_DIR_METHOD="rename"
PREEXISTING_DIR_METHOD="upgrade"

PREDEF_GRID_NAME=GSD_RAP13km
ADDNL_OUTPUT_GRIDS=( "130" "242" )
Expand Down
2 changes: 1 addition & 1 deletion ush/config.sh.RRFS_NA_3km
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ FIX_CRTM=/home/rtrr/FIX_EXEC_MODULE/crtm/CRTM_v2.3.0
VERBOSE="TRUE"

RUN_ENVIR="nco"
PREEXISTING_DIR_METHOD="rename"
PREEXISTING_DIR_METHOD="upgrade"

PREDEF_GRID_NAME=RRFS_NA_3km

Expand Down
4 changes: 2 additions & 2 deletions ush/config.sh.RRFS_dev1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ FIX_GSI=/mnt/lfs4/BMC/nrtrr/RRFS/fix/fix_gsi
FIX_CRTM=/home/rtrr/FIX_EXEC_MODULE/crtm/CRTM_v2.3.0
OBSPATH_NSSLMOSIAC=/public/data/radar/nssl/mrms/conus
RADARREFL_TIMELEVEL=(0 15 30 45)
#FH_DFI_RADAR="0.0,0.25,0.5"
FH_DFI_RADAR="0.0,0.25,0.5"

VERBOSE="TRUE"

RUN_ENVIR="nco"
PREEXISTING_DIR_METHOD="rename"
PREEXISTING_DIR_METHOD="upgrade"
Copy link
Copy Markdown
Collaborator

@christinaholtNOAA christinaholtNOAA May 24, 2021

Choose a reason for hiding this comment

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

Should this not be done for all configs? Seems that we want to have as much consistency across configs as possible for smooth deployment.

Copy link
Copy Markdown
Author

@hu5970 hu5970 May 24, 2021

Choose a reason for hiding this comment

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

We are trying to deploy dev1. I would like to change others when we need to deploy others. This can be a PR from other colleagues. Just want to make RRFS_dev1 deploy simple as we need to catch 22Z cold start.

Copy link
Copy Markdown
Collaborator

@christinaholtNOAA christinaholtNOAA May 24, 2021

Choose a reason for hiding this comment

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

We need to keep these all consistent, especially in a space where everyone is learning. If the proper way to deploy a cycling run and do it smoothly is to use "upgrade" then all the cycling configs should be modified to that now.

Our other colleagues are still learning these things (including me). We were taught to use this setting in the last few sessions as the trusted method, so it should be the default to get a cycling method to run smoothly. Please go ahead and update them.


PREDEF_GRID_NAME=RRFS_CONUS_3km
ADDNL_OUTPUT_GRIDS=( "hrrr" )
Expand Down
2 changes: 1 addition & 1 deletion ush/config.sh.RRFS_wcoss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ SFCOBS_USELIST=${FIX_GSI}
VERBOSE="TRUE"

RUN_ENVIR="nco"
PREEXISTING_DIR_METHOD="rename"
PREEXISTING_DIR_METHOD="upgrade"

PREDEF_GRID_NAME=RRFS_CONUS_3km

Expand Down
3 changes: 3 additions & 0 deletions ush/generate_FV3LAM_wflow.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -847,6 +847,9 @@ if [ "${DO_DACYCLE}" = "true" ]; then
'nggps_ic' : false,
'mountain' : true,
'warm_start' : true,
}
'gfs_physics_nml': {
'fh_dfi_radar': [${FH_DFI_RADAR[@]}],
}"

$USHDIR/set_namelist.py -q \
Expand Down
2 changes: 1 addition & 1 deletion ush/templates/FV3LAM_wflow.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -835,7 +835,7 @@ MODULES_RUN_TASK_FP script.

<dependency>
{%- if do_dacycle and do_nonvar_cldanal%}
<taskdep task="&CLDANL_NONVAR_TN;"/>
<taskdep task="&RADAR_REFL2TTEN_TN;"/>
{%- elif do_dacycle %}
<taskdep task="&ANAL_GSI_TN;"/>
{%- else %}
Expand Down