Skip to content

Commit 1256b5b

Browse files
authored
Merge pull request #1445 from jimmielin/hplin/diag_tke
cam6_4_175: Complete CCPPization of CAM5 UW (diag_TKE) PBL scheme; cleanup to vertical_diffusion_tend
2 parents b7266d2 + b991e83 commit 1256b5b

10 files changed

Lines changed: 582 additions & 4128 deletions

File tree

.gitmodules

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
[submodule "atmos_phys"]
3030
path = src/atmos_phys
3131
url = https://github.com/ESCOMP/atmospheric_physics
32-
fxtag = atmos_phys0_21_003
32+
fxtag = atmos_phys0_22_000
3333
fxrequired = AlwaysRequired
3434
fxDONOTUSEurl = https://github.com/ESCOMP/atmospheric_physics
3535

bld/configure

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2150,6 +2150,7 @@ sub write_filepath
21502150
print $fh "$camsrcdir/src/atmos_phys/schemes/cloud_fraction\n";
21512151
print $fh "$camsrcdir/src/atmos_phys/schemes/vertical_diffusion\n";
21522152
print $fh "$camsrcdir/src/atmos_phys/schemes/holtslag_boville\n";
2153+
print $fh "$camsrcdir/src/atmos_phys/schemes/bretherton_park\n";
21532154

21542155
# Dynamics package and test utilities
21552156
print $fh "$camsrcdir/src/dynamics/$dyn\n";

bld/namelist_files/namelist_definition.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3610,12 +3610,12 @@ Default: 100.e3 (hPa)
36103610
<entry id="eddy_moist_entrain_a2l" type="real" category="pbl"
36113611
group="eddy_diff_nl" valid_values="" >
36123612
Moist entrainment enhancement parameter.
3613-
Default: set by build-namelist
3613+
Default: 30.D0
36143614
</entry>
36153615

36163616
<entry id="kv_top_pressure" type="real" category="pbl"
36173617
group="eddy_diff_nl" valid_values="" >
3618-
Pressure (Pa) that defined the upper atmosphere for adjustment of
3618+
Pressure (Pa) that defines the upper atmosphere for adjustment of
36193619
eddy diffusivities from diag_TKE using kv_top_scale.
36203620
Default: 0.
36213621
</entry>

doc/ChangeLog

Lines changed: 77 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,80 @@
1+
===============================================================
2+
3+
Tag name: cam6_4_175
4+
Originator(s): jimmielin
5+
Date: May 22, 2026
6+
One-line Summary: Complete CCPPization of CAM5 UW (diag_TKE) PBL scheme; cleanup to vertical_diffusion_tend
7+
Github PR URL: https://github.com/ESCOMP/CAM/pull/1445
8+
9+
Purpose of changes (include the issue number and title text for each relevant GitHub issue):
10+
- Closes #1429 - Move University of Washington moist turbulence scheme ("diag_TKE", CAM5 PBL scheme) to atmos_phys.
11+
- Cleanup to vertical_diffusion_tend to accommodate CCPPization of HB (CAM4) and UW (CAM5) PBL schemes.
12+
- Minor fixes to snapshot metadata.
13+
14+
Describe any changes made to build system: added bretherton_park folder to atmos_phys
15+
16+
Describe any changes made to the namelist: update to descriptions only.
17+
18+
List any changes to the defaults for the boundary datasets: N/A
19+
20+
Describe any substantial timing or memory changes: None
21+
22+
Code reviewed by: nusbaume, cacraig, peverwhee (atmos_phys)
23+
24+
List all files eliminated:
25+
```
26+
D src/physics/cam/eddy_diff.F90
27+
- moved to atmos_phys
28+
29+
D src/physics/cam/pbl_utils.F90
30+
- only remaining subroutine now in atmos_phys bretherton_park_diff.F90
31+
```
32+
33+
List all files added and what they do:
34+
35+
List all existing files that have been modified, and describe the changes:
36+
```
37+
M bld/configure
38+
- add bretherton_park subdir for atmos_phys
39+
40+
M bld/namelist_files/namelist_definition.xml
41+
- fix description
42+
43+
M src/control/cam_snapshot_common.F90
44+
- populate units for pbuf_ksrftms, pbuf_WSEDL
45+
46+
M src/physics/cam/eddy_diff_cam.F90
47+
- moved most of existing logic in eddy_diff_tend to new CCPP scheme bretherton_park_diff_run
48+
- what remains is just CAM-specific code for history and code to interface with the underlying run phase.
49+
50+
M src/physics/cam/vertical_diffusion.F90
51+
- move kinematic fluxes and obklen calculation to common interstitial scheme.
52+
- move input preparation (vert coord for vdiff, potential temp) to common interstitial scheme.
53+
- rearrange data flow for HB, UW to share as much code as possible.
54+
```
55+
56+
If there were any failures reported from running test_driver.sh on any test
57+
platform, and checkin with these failures has been OK'd by the gatekeeper,
58+
then copy the lines from the td.*.status files for the failed tests to the
59+
appropriate machine below. All failed tests must be justified.
60+
61+
derecho/intel/aux_cam: All PASS
62+
63+
derecho/nvhpc/aux_cam: All PASS
64+
65+
izumi/nag/aux_cam:
66+
ERC_D_Ln9.f10_f10_mt232.FHIST_C5.izumi_nag.cam-outfrq3s_subcol (Overall: FAIL)
67+
- pre-existing failure -- see https://github.com/ESCOMP/CAM/issues/1514
68+
69+
izumi/gnu/aux_cam: All PASS
70+
71+
CAM tag used for the baseline comparison tests if different than previous
72+
tag:
73+
74+
Summarize any changes to answers: All B4B
75+
76+
===============================================================
77+
178
Tag name: cam6_4_174
279
Originator(s): kuanchihwang
380
Date: 21 May 2026

src/atmos_phys

Submodule atmos_phys updated 42 files

src/control/cam_snapshot_common.F90

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1442,7 +1442,7 @@ subroutine fill_pbuf_info(pbuf_info, pbuf, const_cname)
14421442
'ipbl ','unset ',&
14431443
'ISS_FRAC ','unset ',&
14441444
'kpblh ','unset ',&
1445-
'ksrftms ','unset ',&
1445+
'ksrftms ','kg/m2/s ',&
14461446
'kvh ','m2/s ',&
14471447
'kvm ','m2/s ',&
14481448
'kvt ','m2/s ',&
@@ -1606,7 +1606,7 @@ subroutine fill_pbuf_info(pbuf_info, pbuf, const_cname)
16061606
'wpthlp_mc_zt ','unset ',&
16071607
'WPTHLP_nadv ','unset ',&
16081608
'WPTHVP ','unset ',&
1609-
'WSEDL ','unset ',&
1609+
'WSEDL ','m/s ',&
16101610
'wstarPBL ','unset ',&
16111611
'ZM_DP ','unset ' /), (/2,100/))
16121612

0 commit comments

Comments
 (0)