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

Two FV3 dycore bug fixes and SGS cloud update gsd/develop 2020/06/29 #32

Conversation

climbfuji
Copy link

@climbfuji climbfuji commented Jun 29, 2020

This PR:

  • updates the submodule pointers for fv3atm, GFDL_atmos_cubed_sphere and ccpp-physics for the changes described in the PRs below
  • reactivates the regression test fv3_ccpp_gsd_sar_25km_debug (issue Reactivate regression test fv3_ccpp_gsd_sar_25km_debug #31)
  • includes a formal update from NOAA-EMC (which only contains parts of the changes described above).

Associated PRs:
NOAA-GSL/ccpp-physics#42
NOAA-GSL/GFDL_atmos_cubed_sphere#5
NOAA-GSL/fv3atm#39
#32

For regression testing information, see below.

@climbfuji climbfuji changed the title PLACEHOLDER Dycore bugfixes and sgscloud update gsd develop 20200629 Two FV3 dycore bug fixes and SGS cloud update gsd/develop 2020/06/29 Jun 29, 2020
…fs-community#155)

* Update .gitmodules and submodule pointer for fv3atm for code review and testing

* Regression test logs for hera.intel and hera.gnu

* Regression test logs for orion.intel

* Regression test logs for wcoss_cray and wcoss_dell_p3

* Revert change to .gitmodules and update submodule pointer for fv3atm
…r-model into dycore_bugfixes_and_sgscloud_update_gsd_develop_20200629
@climbfuji climbfuji marked this pull request as ready for review June 29, 2020 17:20
@climbfuji climbfuji requested a review from DomHeinzeller as a code owner June 29, 2020 17:20
@climbfuji
Copy link
Author

Regression testing on hera.intel against official EMC baseline using rt.conf. The following tests fail as expected (but run to completion), all others pass.

fv3_ccpp_gsd_debug
fv3_ccpp_thompson_no_aero_debug
fv3_ccpp_thompson_debug
fv3_ccpp_thompson
fv3_ccpp_thompson_no_aero
fv3_ccpp_gsd

rt_full.log
rt_full_fail_test.log

@climbfuji
Copy link
Author

@tanyasmirnova @joeolson42 @hannahcbarnes FYI - the SGS updates from Tanya and two bugfixes for the dycore. Will merge as soon as the rt_ccpp_gsd.conf tests finished successfully.

@climbfuji
Copy link
Author

Regression testing on hera.intel using rt_ccpp_gsd.conf (first create, then verify). All tests pass, including the re-activated fv3_ccpp_gsd_sar_25km_debug.

rt_ccpp_gsd_create.log
rt_ccpp_gsd_verify.log

Copy link

@DomHeinzeller DomHeinzeller left a comment

Choose a reason for hiding this comment

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

Still need to run the tests on hera.gnu ...

@DomHeinzeller
Copy link

Regression testing on hera.gnu against official EMC baseline using rt_gnu.conf. The following tests fail as expected (but run to completion), all others pass.

fv3_ccpp_thompson
fv3_ccpp_thompson_no_aero
fv3_ccpp_gsd

rt_full_gnu.log
rt_full_gnu_fail_test.log

@DomHeinzeller DomHeinzeller merged commit 27a8a07 into NOAA-GSL:gsd/develop Jun 29, 2020
@hannahcbarnes
Copy link

I will start a retro and have the results for next week’s meeting

@hannahcbarnes
Copy link

hannahcbarnes commented Jun 29, 2020 via email

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