-
Notifications
You must be signed in to change notification settings - Fork 29
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
Adding new vertical levels and mpasout etc #565
base: rrfs-mpas-jedi
Are you sure you want to change the base?
Conversation
Modified the namelist files for conus3km using the hrrrv5 physics (consistant with current GSL MPAS runs) Change hard-coded ztop and nvertlevels to the settings from zeta_levels Add mpasout outputs, along with invariant.nc as input Remove ${MESH_NAME} from parm/streams.atmosphere_fcst
config_thompson_hail_aware = true | ||
config_convection_scheme = 'off' | ||
! config_physics_suite = '${physics_suite}' | ||
config_physics_suite = 'hrrrv5' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- We need to use line 56 and define correct
physics_suite
inexrrfs_fcst.sh
- To confirm, do we have a
hrrrv5
physics suite ready at this moment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the namelist that works for the current GSL MPAS model, consistent with the GSL real-time runs. My previous tests using "convection_permitting" would crash. For details we need inputs from the physics people.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chunhuazhou Thanks for the reply. We better NOT hardcode this to "hrrrv5", we can generate this on-the-fly in the exrrfs-fcst.sh
script. So remove line 57, revert line 56, and specify "hrrrv5" in the script.
For the physics suite, I did not run the new GSL version of MPAS, so we will need @hu5970's input on this.
@@ -44,6 +44,7 @@ | |||
&preproc_stages | |||
config_static_interp = false | |||
config_native_gwd_static = false | |||
config_native_gwd_gsl_static = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to set up lines 46 and 47 on the fly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is good to be like this, suggested by Mike Toy when I was having issues running the ic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we don't use gsl_static, why do we bother to add this namelist option?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The default value is true but it might be only used when generating static.nc
So for ic and lbc tasks, it is better to set it to false.
@@ -33,7 +33,8 @@ | |||
/ | |||
&printout | |||
config_print_global_minmax_vel = true | |||
config_print_detailed_minmax_vel = false | |||
config_print_detailed_minmax_vel = true | |||
config_print_global_minmax_sca = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lines 36 and 37: Do we want to set them to true for all rrfs runs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are useful printouts. I think it is good to add them.
@@ -13,11 +13,11 @@ | |||
config_nfgsoillevels = ${nfgsoillevels} | |||
/ | |||
&data_sources | |||
config_geog_data_path = '' | |||
config_geog_data_path = '/mnt/lfs5/BMC/wrfruc/HRRRv5/geog/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
config_geog_data_path
can be empty in the rrfs-workflow
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this to be consistent with what is set for the "convection_permitting". Can't be removed if desired.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, let's keep it as an empty string.
Also, to be consistent, let's remove the hardwired PATH in convection_permitting
as well. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted. Thanks!
config_met_prefix = '${prefix}' | ||
config_sfc_prefix = 'SST' | ||
config_fg_interval = ${interval_seconds} | ||
config_landuse_data = 'MODIFIED_IGBP_MODIS_NOAH' | ||
config_landuse_data = 'MODIFIED_IGBP_MODIS_NOAH_15s' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is under parm/mesoscal_reference
. Do we have to modify line 20?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this to be consistent with what is set for the "convection_permitting". Can't be reverted if desired.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep the original version. We can revisit this if we have to make this change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted. Thanks!
DESCRIPTION OF CHANGES:
Other changes to some fix/static files are not committed here, due to filesizes, including:
- fix/meshes/conus12km.static.nc
- fix/meshes/conus3km.static.nc
- fix/meshes/conus12km.invariant.nc_L65
- fix/meshes/conus12km.ugwp_oro_data.nc
- fix/meshes/conus3km.invariant.nc_L62
- fix/meshes/conus3km.invariant.nc_L65
- fix/meshes/conus3km.ugwp_oro_data.nc
- fix/physics/convection_permitting/MP_TEMPO_HAILAWARE_QRacrQG_DATA.DBL
- fix/physics/convection_permitting/MP_TEMPO_QIautQS_DATA.DBL
- fix/physics/convection_permitting/MP_TEMPO_QRacrQS_DATA.DBL
- fix/physics/convection_permitting/MP_TEMPO_freezeH2O_DATA.DBL
- fix/physics/convection_permitting/MP_THOMPSON_QIautQS_DATA.DBL
- fix/physics/convection_permitting/MP_THOMPSON_QRacrQG_DATA.DBL
- fix/physics/convection_permitting/MP_THOMPSON_QRacrQS_DATA.DBL
- fix/physics/convection_permitting/MP_THOMPSON_freezeH2O_DATA.DBL
CONTRIBUTORS (optional):