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

release/public-v2: noahmp_bugfix #393

Conversation

llpcarson
Copy link
Collaborator

Description

(1) Some updates to prevent the model from crashing.
(2) Include opt_stc=3 bug fix in both noahmp land and glacier code (no initialization for some parameters)
(3) The wrong downward longwave forcing to Noah MP. Noah MP expects the
downward longwave before surface emission, however the current driver passed the
one after emission. Inside Noah MP it is multiplied by surface emissivity
again. So generally Noah MP is getting less downward longwave.

Issue(s) addressed

NCAR/ccpp-physics#556

Testing

RT completed for cheyenne.intel, cheyenne.gnu and hera.intel
Baseline changed for all tests using NOAH-MP (control, RRFS)
New baselines are:
hera.intel: /scratch1/BMC/gmtb/stmp4/Laurie.Carson/FV3_RT/REGRESSION_TEST_RELEASE_PUBLIC_V2_INTEL/
cheyenne.intel: /glade/work/carson/FV3_RT/REGRESSION_TEST_RELEASE_PUBLIC_V2_INTEL
cheyenne.gnu: /glade/work/carson/FV3_RT/REGRESSION_TEST_RELEASE_PUBLIC_V2_GNU

Dependencies

@climbfuji
Copy link
Collaborator

@llpcarson I assume I should copy the baselines you created into the final location. Can you change the date tag in rt.sh (RTPWD) please? Shall we use 20210126? I will run the regression tests on the three systems afterwards to verify that everything is in the right place, and I will create baselines on the other platforms, too. Thanks!

@llpcarson
Copy link
Collaborator Author

Done, thanks! Let me know if there are any permissions issues.

@climbfuji
Copy link
Collaborator

@llpcarson please copy all log files from cheyenne for this PR. I reran the tests on cheyenne and hera after moving the data:

heinzell@cheyenne1:/glade/work/heinzell/fv3/ufs-weather-model/ufs-weather-model-public_release_v5_noahmp_bugfix/gnu/tests [gnu|P48503002]> ls -l /glade/work/heinzell/fv3/ufs-weather-model/ufs-weather-model-public_release_v5_noahmp_bugfix/logs_for_laurie/
total 192
-rw-r--r-- 1 heinzell ncar 26635 Jan 27 08:19 RegressionTests_cheyenne.gnu.log
-rw-r--r-- 1 heinzell ncar 26661 Jan 27 08:19 RegressionTests_cheyenne.intel.log
-rw-r--r-- 1 heinzell ncar 26551 Jan 27 10:55 RegressionTests_gaea.intel.log
-rw-r--r-- 1 heinzell ncar 26791 Jan 27 14:16 RegressionTests_hera.gnu.log
-rw-r--r-- 1 heinzell ncar 26564 Jan 27 09:00 RegressionTests_hera.intel.log
-rw-r--r-- 1 heinzell ncar 26512 Jan 27 14:17 RegressionTests_jet.intel.log

Thanks!

@llpcarson
Copy link
Collaborator Author

llpcarson commented Jan 27, 2021 via email

@climbfuji
Copy link
Collaborator

Done

Great - last bit, update of the fv3atm submodule pointer and reversal of the .gitmodules change, please.

@llpcarson
Copy link
Collaborator Author

llpcarson commented Jan 27, 2021 via email

Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Thanks for making those changes!

@climbfuji climbfuji merged commit 0fd19cf into ufs-community:release/public-v2 Jan 27, 2021
@llpcarson llpcarson deleted the public_release_v5_noahmp_bugfix branch March 24, 2021 15:14
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