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

MYNN-EDMF updates and bug fixes #97

Merged

Conversation

climbfuji
Copy link

@climbfuji climbfuji commented Aug 4, 2021

Description

Update .gitmodules and submodule pointer for fv3atm for code review and testing of the changes in NOAA-GSL/ccpp-physics#102 (MYNN-EDMF updates and bug fixes) from @joeolson42.

Testing

  1. On hera with Intel, create new baselines using rt_ccpp_dev.conf and then verify against them - all tests pass.
    rt_ccpp_dev_hera_intel_create.log
    rt_ccpp_dev_hera_intel_verify.log

  2. On hera with GNU, create new baselines using rt_ccpp_dev.conf and then verify against them - all tests pass.
    rt_ccpp_dev_hera_gnu_create.log
    rt_ccpp_dev_hera_gnu_verify.log

  3. On hera with Intel, verify against existing baselines and make sure that all tests run to completion and only those fail with b4b mismatches that we expect to fail - correct.
    rt_hera_intel_verify_against_existing.log
    rt_hera_intel_verify_against_existing_fail_test.log

  4. On hera with GNU, verify against existing baselines and make sure that all tests run to completion and only those fail with b4b mismatches that we expect to fail - correct.
    rt_hera_gnu_verify_against_existing.log
    rt_hera_gnu_verify_against_existing_fail_test.log

Dependencies

NOAA-GSL/ccpp-physics#102
NOAA-GSL/fv3atm#103
#97

@DomHeinzeller
Copy link

The fv3atm hash is correct, this PR has no changes, merging now.

@DomHeinzeller DomHeinzeller merged commit 7c61af8 into NOAA-GSL:gsl/develop Aug 11, 2021
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.

2 participants