-
Notifications
You must be signed in to change notification settings - Fork 14
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Implements cam_thermo_water_update and CCPPized check_energy (#316)
Originator(s): jimmielin Description (include the issue title, and the keyword ['closes', 'fixes', 'resolves'] followed by the issue number): All changes are bit-for-bit, except those noted: Implements `cam_thermo_water_update`: - updates `cp_or_cv_dycore` (`specific_heat_for_air_used_in_dycore`) in `air_composition.F90` for SE and MPAS dynamical cores - read `cp_or_cv_dycore` from CAM snapshot (refer to companion CAM PR) - energy formula (that has to be matching dycore) is recognized and set by null-dycore in `dyn_grid.F90` by looking at global attributes of the initial file; see `find_energy_formula` - added `is_first_timestep` logical state flag Ports `cam_thermo` and related updates in `air_composition` and `dp_coupling` from CAM 6.3.109 (https://github.com/ESCOMP/CAM/pull/761/files) - update to hydrostatic energy calculation - changes `get_cp`, `get_R` in `air_composition.F90` to use moist mixing ratios - **answer-changing:** update to moist-to-dry (for physics) conversion in `dp_coupling::derived_phys_dry` to account for all water tracers instead of just Q - **answer-changing:** update to not-really-"exner" calculation to use composition-dependent `cappav` instead of `cappa` Changes `vcoord` in `dyn_tests_utils` (old CAM) to `energy_formula` now in `cam_thermo_formula` (separated out into a different file to avoid dependency issues) - `vc_moist_pressure` is now `ENERGY_FORMULA_DYCORE_FV`; `vc_dry_pressure` is `_SE`; `vc_height` is `_MPAS` - these are just integer flags (0,1,2) and values are kept consistent with old CAM and their use in dynamics tests Ports global mean utility module (`gmean_mod.F90`), de-chunkized from CAM: - Implements `get_wght` in `physics_grid` for weighted sum calculation Imports `check_energy_chng` and `check_energy_fix` from `atmospheric_physics` Describe any changes made to build system: N/A Describe any changes made to the namelist: contained within ncar-physics List any changes to the defaults for the input datasets (e.g. boundary datasets): - Added `cp_or_cv_dycore` in CAM snapshots List all files eliminated and why: N/A List all files added and what they do: ``` - energy_formula A src/data/cam_thermo_formula.F90 A src/data/cam_thermo_formula.meta - gmean A src/utils/gmean_mod.F90 ``` List all existing files that have been modified, and describe the changes: (Helpful git command: `git diff --name-status development...<your_branch_name>`) ``` - ncar-physics update M .gitmodules M src/physics/ncar_ccpp - `is_first_timestep` M src/control/cam_comp.F90 - cam_thermo_water_update for cp_or_cv_dycore (include in registry; read from ic) M src/data/air_composition.F90 M src/data/cam_thermo.F90 M src/data/registry.xml M tools/stdnames_to_inputnames_dictionary.xml M src/dynamics/se/dp_coupling.F90 M src/dynamics/se/dycore/prim_advance_mod.F90 M src/dynamics/se/dyn_comp.F90 M src/dynamics/utils/dyn_thermo.F90 - energy_formula M src/physics/utils/phys_comp.F90 M src/dynamics/mpas/dyn_comp.F90 M src/dynamics/none/dyn_comp.F90 M src/dynamics/none/dyn_grid.F90 - gmean M src/physics/utils/physics_grid.F90 ``` Note: bit-for-bit in check_energy with CAM is tricky to validate without dycore updates to SE; may need to merge #301 first --------- Co-authored-by: Kuan-Chih Wang <[email protected]>
- Loading branch information
1 parent
edf84cd
commit 37fdbfb
Showing
20 changed files
with
955 additions
and
233 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.