Add prognostic cumulus closure description in saSAS #523
Conversation
|
@lisa-bengtsson NCAR/ccpp-physics#903 has been merged. Please pull down the latest ccpp-physics main branch, git add that latest branch, and revert your changes to .gitmodules in preparation for merging this PR. |
|
I have added the NCAR/ccpp-physics#903 and reverted .gitmodules, I still see the changes in ccpp/physics here (in fv3atm), how can I see that it is pointing to the correct ccpp/physics? |
@lisa-bengtsson You should see that there are changes in ccpp/physics because you've updated the ccpp-physics submodule. The latest ccpp-physics commit hash is NCAR/ccpp-physics@77bcfb1 and it looks like your branch is correctly pointing to it, so we're all good. You can check this on GitHub by looking at the 'files changed' in the PR, clicking on the ccpp/physics and then the number of changed files (19 files). It'll show which commit has fv3atm is pointing to. You can also use git in the command line to do this. Assuming that your latest fv3atm branch is checked out, if you do a I'm sure that there are other ways as well. Edit: I just googled, and you can do |
Great, thank you Grant. Then I think we are good to go with this branch. |
…OAA-EMC#523" https://github.com/NOAA-EMC/fv3atm/pull/523/files modified: ../ccpp/data/CCPP_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.meta modified: ../ccpp/driver/GFS_restart.F90 modified: ../ccpp/physics (new commits, untracked content)
…OAA-EMC#523" https://github.com/NOAA-EMC/fv3atm/pull/523/files modified: ../ccpp/data/CCPP_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.meta modified: ../ccpp/driver/GFS_restart.F90 modified: ../ccpp/physics (new commits, untracked content) modified: ../ccpp/data/CCPP_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.meta modified: ../ccpp/driver/GFS_restart.F90 modified: ../ccpp/physics (new commits, untracked content)
…OAA-EMC#523" https://github.com/NOAA-EMC/fv3atm/pull/523/files modified: ../ccpp/data/CCPP_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.meta modified: ../ccpp/driver/GFS_restart.F90 modified: stochastic_physics_wrapper.F90
…OAA-EMC#523" (#1) https://github.com/NOAA-EMC/fv3atm/pull/523/files modified: ../ccpp/data/CCPP_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.meta modified: ../ccpp/driver/GFS_restart.F90 modified: stochastic_physics_wrapper.F90
* Adopt progsigma "Add prognostic cumulus closure description in saSAS NOAA-EMC#523" https://github.com/NOAA-EMC/fv3atm/pull/523/files modified: ../ccpp/data/CCPP_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.meta modified: ../ccpp/driver/GFS_restart.F90 modified: stochastic_physics_wrapper.F90 * modified: .gitmodules
* Adopt progsigma "Add prognostic cumulus closure description in saSAS NOAA-EMC#523" https://github.com/NOAA-EMC/fv3atm/pull/523/files modified: ../ccpp/data/CCPP_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.F90 modified: ../ccpp/data/GFS_typedefs.meta modified: ../ccpp/driver/GFS_restart.F90 modified: stochastic_physics_wrapper.F90 * modified: .gitmodules * modified: ccpp/physics
Description
This PR introduces an optional closure assumption to the Arakawa-Schubert cloud work function in saSAS cumulus convection, as well as the GFS shallow convection scheme, in where the mass-flux at cloud base is described as the product between updraft area fraction and updraft velocity. The updraft area fraction is given by a prognostic moisture budget equation.
The update is not aimed for UFS coupled prototype 8, but is an option to be explored beyond P8.
In addition to updates in the ccpp/physics submodule, new field_table, and namelist parameters are added in ufs-weather-model.
The PR is also making a small change in the stochastic_physics_driver.F90 to allow the cellular automata to work on non-uniform blocksizes. Previously the code aborted if not all blocks in the domain had the same size, however, the code works the way it's been formulated, so I simply removed the call to mpp_error in case on non-uniform blocksizes.
Issue(s) addressed
Issue: NCAR/ccpp-physics#890
ufs-community/ufs-weather-model#1193
Testing
This PR does not change the answers in the control, but a new test has been added where the scheme is activated using progsigma = True.
The regression tests have been tested on Hera.
Dependencies
Dependency:
NCAR/ccpp-physics#903
ufs-community/ufs-weather-model#1187