Skip to content

Thompson subcycling for develop#595

Closed
climbfuji wants to merge 16 commits into
ufs-community:developfrom
climbfuji:thompson_subcycling_for_develop
Closed

Thompson subcycling for develop#595
climbfuji wants to merge 16 commits into
ufs-community:developfrom
climbfuji:thompson_subcycling_for_develop

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented May 24, 2021

PR Checklist

  • Ths PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR. Please consult the ufs-weather-model wiki if you are unsure how to do this.

  • This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR

  • An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
    are specified below.

  • If new or updated input data is required by this PR, it is clearly stated in the text of the PR.

Description

Implement a subcycling capability for Thompson MP in CCPP and exercise it in the UFS regression tests. This ufs-weather-model PR only updates the submodule pointer for fv3atm for the changes w.r.t. Thompson MP described in the associated PRs below.

Additional changes:

  • add missing modulefile ufs_hera.gnu_debug
  • bugfix in compile.sh to copy the correct modulefile when DEBUG is used

The changes to the suite FV3_GSD_noah/FV3_GFS_v16_thompspn (run Thompson MP with 4/2 subcycles) change the answer of several Thompson MP based regression tests. No new input data required.

Issue(s) addressed

Fixes #596

Testing

Preliminary regression testing on Hera with GNU and Intel against existing baseline (2021/05/26): all tests that are expected to pass do pass, and all tests that are expected to fail do fail with b4b mismatches (but they all run to completion):

rt_hera_gnu_verify_against_existing.log
rt_hera_gnu_verify_against_existing_fail_test.log

rt_hera_initel_verify_against_existing.log
rt_hera_initel_verify_against_existing_fail_test.log

Full regression tests will be run on all tier-1 platforms when it is time to commit.

  • hera.intel
  • hera.gnu
  • orion.intel
  • cheyenne.intel
  • cheyenne.gnu
  • gaea.intel
  • jet.intel
  • wcoss_cray
  • wcoss_dell_p3

Dependencies

NCAR/ccpp-physics#666
NCAR/ccpp-framework#376
NOAA-EMC/ufsatm#316
#595

@climbfuji climbfuji marked this pull request as ready for review May 26, 2021 17:27
@climbfuji
Copy link
Copy Markdown
Collaborator Author

This PR was replaced with #632.

@climbfuji climbfuji closed this Jun 10, 2021
zhanglikate pushed a commit to zhanglikate/ufs-weather-model that referenced this pull request Oct 20, 2025
…-community#595)

* Change the format of domain restart files to netcdf4 and set chunksizes

* Remove nc_format="netcdf4" argument when opening restart files

* Set chunksize of zaxis and time axis to 1

* Update clm_lake and rrfs_sd modules to support chunksizes

* Make quilting restart files identical to fms files
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.

Implement subcycling capability for Thompson microphysics

3 participants