-
Notifications
You must be signed in to change notification settings - Fork 7
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
HWRF RRTMG #30
HWRF RRTMG #30
Conversation
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.
Again, no need to make changes, but please look at the comments. The new test also need to be added to an existing regression test config (e.g. rt.conf
) - will do.
/ | ||
|
||
&fv_core_nml | ||
layout = 24,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.
This will need to be changed to 20,24 to avoid non-uniform block sizes (we should also check all other regional_c768 tests that we set up recently).
@@ -0,0 +1,31 @@ | |||
rsync -arv @[RTPWD]/FV3_regional_input_data/. . | |||
rsync -arv @[RTPWD]/@[INPUT_DIR]/model_configure . |
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 will not work - need to change @INPUT_DIR
to fv3_regional_c768
rm -rf INPUT RESTART | ||
mkdir INPUT RESTART | ||
|
||
rsync -arv @[RTPWD]/@[INPUT_DIR]/INPUT/. INPUT/. |
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 will not work - need to change @INPUT_DIR
to fv3_regional_c768
@@ -0,0 +1,85 @@ | |||
############################################################################### | |||
# | |||
# FV3 CCPP GFS v15.2 compiled with 32-bit dynamics test |
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.
Update description
# | ||
############################################################################### | ||
|
||
export TEST_DESCR="Compare FV3 32bit CCPP GFS HWRF RRTMG results with previous trunk version" |
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.
Is the test really compiled with 32-bit dynamics?
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.
no. I copied a old template.
|
||
export TEST_DESCR="Compare FV3 CCPP regional c768 with Ferrier-Aligo MP scheme results with previous trunk version" | ||
|
||
export CNTL_DIR=fv3_regional_c768 |
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 needs to be renamed to fv3_regional_c768_fa_hwrf_rrtmg
@mzhangw Please copy file /home/Man.Zhang/diag_table_HWRF to a place somewhere on scratch so that I can add it to the regression test input directories. Thank you! |
/scratch1/BMC/gmtb/Man.Zhang/diag_table_HWRF
… On Apr 7, 2020, at 10:42 AM, Dom Heinzeller ***@***.***> wrote:
@mzhangw <https://github.com/mzhangw> Please copy file /home/Man.Zhang/diag_table_HWRF to a place somewhere on scratch so that I can add it to the regression test input directories. Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#30 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG7TW2QWSFSDYN54G245XLDRLNJYJANCNFSM4LS3JCXA>.
|
Thanks, copied it - you can delete the file. |
Why does the ccpp_hwrf_rrtmg.nml.IN not have the HWRF RRTMG switches?
|
Which run should I be comparing the test |
those are a previous test with GFDL mp.
For HWRF physics, we need a bundle of FA and HWRF RRTMG, so we only need to keep
parm/ccpp_regional_c768_FA_HRRTMG.nml.IN
tests/fv3_conf/ccpp_regional_FA_HRRTMG_run.IN
tests/tests/fv3_ccpp_regional_c768_FA_HRRTMG
… On Apr 7, 2020, at 11:29 AM, Dom Heinzeller ***@***.***> wrote:
Why does the ccpp_hwrf_rrtmg.nml.IN not have the HWRF RRTMG switches?
...
icloud = 3
iovr_lw = 4
iovr_sw = 4
spec_adv = .true.
RHGRD = 0.98
...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#30 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG7TW2UXACAVHIIRGYJFU5TRLNPJBANCNFSM4LS3JCXA>.
|
Ok, thanks for the clarification. Will remove the leftover stuff from your PR. |
@mzhangw @grantfirl @ligiabernardet @llpcarson I ran the regression test The forecast completed successfully, but I am seeing the following errors/warnings in file
|
|
Ok thanks. But you did see this error messages? |
Yes
… On Apr 7, 2020, at 1:51 PM, Dom Heinzeller ***@***.***> wrote:
This WARN message is harmless per email communication with @ericaligo-NOAA <https://github.com/ericaligo-NOAA> @ChunxiZhang-NOAA <https://github.com/ChunxiZhang-NOAA>
I do not have previous run currently.
Ok thanks. But you did see this error messages?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#30 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG7TW2TXBJCFCNUP6VOPF4LRLN75HANCNFSM4LS3JCXA>.
|
well, I found one previous run in my march sandbox: |
Thanks, I copied this over as well so that I can do a comparison with the "normal" FA run. |
dtc/hwrf-physics: HWRF RRTMG (based on #30)
This PR was merged into dtc/hwrf-physics, see #36. Keeping this PR open until dtc/hwrf-physics is merged into EMC develop. |
@mzhangw Please confirm that this PR can be closed. Thanks! |
This PR can be deleted. |
…_emc_20200625 Update gsd/develop from dev/emc 2020/06/25 and mods to MYNN sfc and PBL for fractional/coupled
… EARTH_GRID_COMP to UFSDriver (ufs-community#1239) * Require specific package version in top-level CMakeLists.txt * Rename EARTH_GRID_COMP to UFSDriver * Update s4 support (NCAR#31) * Updated S4 compiler versions ufs-community#1223 * Single dt output (NCAR#30)
HWRF RRTMG configuration