Skip to content

Commit 77b1773

Browse files
authored
Merge pull request #1460 from cacraigucar/cam_gravity_bugfix
cam6_4_136: bring in atmos_phys with gravity wave bug fix
2 parents 2a52c8a + c626aa7 commit 77b1773

File tree

3 files changed

+91
-2
lines changed

3 files changed

+91
-2
lines changed

.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_19_000
32+
fxtag = atmos_phys0_19_001
3333
fxrequired = AlwaysRequired
3434
fxDONOTUSEurl = https://github.com/ESCOMP/atmospheric_physics
3535

doc/ChangeLog

Lines changed: 89 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,92 @@
1+
2+
===============================================================
3+
4+
Tag name: cam6_4_136
5+
Originator(s): JulioTBacmeister cacraigucar
6+
Date: Dec 17, 2025
7+
One-line Summary: Bring in atmos_phys with gravity wave bug fix
8+
Github PR URL: https://github.com/ESCOMP/CAM/pull/1460
9+
10+
Purpose of changes (include the issue number and title text for each relevant GitHub issue):
11+
- Implement momentum conservation for moving mountain gravity waves: https://github.com/ESCOMP/CAM/issues/1451
12+
13+
Describe any changes made to build system: N/A
14+
15+
Describe any changes made to the namelist: N/A
16+
17+
List any changes to the defaults for the boundary datasets: N/A
18+
19+
Describe any substantial timing or memory changes: N/A
20+
21+
Code reviewed by: nusbaume peverwhee (for atmos_phys changes)
22+
23+
List all files eliminated: N/A
24+
25+
List all files added and what they do: N/A
26+
27+
List all existing files that have been modified, and describe the changes:
28+
M .gitmodules
29+
M src/atmos_phys
30+
- Bring in updated atmos_phys tag
31+
- Mods were in gravity_wave_drag_moving_mountain.F90
32+
33+
If there were any failures reported from running test_driver.sh on any test
34+
platform, and checkin with these failures has been OK'd by the gatekeeper,
35+
then copy the lines from the td.*.status files for the failed tests to the
36+
appropriate machine below. All failed tests must be justified.
37+
38+
derecho/intel/aux_cam:
39+
ERC_D_Ln9.mpasa120_mpasa120.FHISTC_LTso.derecho_intel.cam-outfrq9s_mpasa120 (Overall: DIFF) details:
40+
ERC_D_Ln9.mpasa120_mpasa120.QPC7.derecho_intel.cam-outfrq9s_mpasa120 (Overall: DIFF) details:
41+
ERC_D_Ln9.ne30pg2_ne30pg2_mt232.QPC7.derecho_intel.cam-outfrq9s (Overall: DIFF) details:
42+
ERC_D_Ln9.ne30pg3_ne30pg3_mt232.F1850C_LTso.derecho_intel.cam-outfrq9s (Overall: DIFF) details:
43+
ERI_D_Ln18.ne30pg3_ne30pg3_mt232.FHISTC_LTso.derecho_intel.cam-outfrq3s_eri (Overall: DIFF) details:
44+
ERP_D_Ln9.ne30pg3_ne30pg3_mt232.F1850C_MTso.derecho_intel.cam-outfrq9s (Overall: DIFF) details:
45+
ERP_Ld3.ne16pg3_ne16pg3_mg17.FHISTC_WAt1ma.derecho_intel.cam-reduced_hist1d (Overall: DIFF) details:
46+
ERP_Ld3.ne30pg3_ne30pg3_mt232.FHISTC_MTt4s.derecho_intel.cam-outfrq1d_aoa (Overall: DIFF) details:
47+
ERP_Ln9.ne30pg3_ne30pg3_mg17.FHISTC_WAma.derecho_intel.cam-outfrq9s (Overall: DIFF) details:
48+
ERR_Ln9.ne16pg3_ne16pg3_mt232.FHISTC_LTso.derecho_intel.cam-outfrq9s_bwic (Overall: DIFF) details:
49+
ERS_Ln9.ne30pg3_ne30pg3_mg17.FHISTC_WXma.derecho_intel.cam-outfrq9s_ctem (Overall: DIFF) details:
50+
SMS_C2_D_Ln9.ne16pg3_ne16pg3_mg17.FHISTC_WXma.derecho_intel.cam-outfrq9s (Overall: DIFF) details:
51+
SMS_D_Ln9.ne30pg3_ne30pg3_mt232.FHISTC_MTso.derecho_intel.cam-outfrq9s (Overall: DIFF) details:
52+
SMS_D_Ln9_P1280x1.ne30pg3_ne30pg3_mt232.FHISTC_MTt1s.derecho_intel.cam-outfrq9s_Leung_dust (Overall: DIFF) details:
53+
- Answer changes expected due to moving mountain bug fix
54+
55+
SMS_D_Ln9_P1536x1.ne0CONUSne30x8_ne0CONUSne30x8_mt12.FCHIST.derecho_intel.cam-outfrq9s (Overall: DIFF) details:
56+
- intermittent failure in CTSM code (lnd_set_decomp_and_domain.F90)
57+
- This test passed on the first try during this run.
58+
59+
derecho/nvhpc/aux_cam:
60+
FAIL ERS_Ln9.ne30pg3_ne30pg3_mt232.FHISTC_LTso.derecho_nvhpc.cam-outfrq9s_gpu_default
61+
-pre-existing failure -- exceeds wallclock time limit
62+
NOTE: Jian has determined this is due to changes on derecho with the the last upgrade.
63+
He has reported the issue to CISL. Note cam6_4_128 is the last CAM tag with
64+
baselines to use for comparison but answer changes are expected starting with
65+
cam6_4_130.
66+
67+
izumi/nag/aux_cam:
68+
ERC_D_Ln9.mpasa480_mpasa480_mt232.QPC7.izumi_nag.cam-outfrq9s_mpasa480 (Overall: DIFF) details:
69+
ERC_D_Ln9.ne3pg3_ne3pg3_mt232.FHISTC_LTso.izumi_nag.cam-cosp_rad_diags (Overall: DIFF) details:
70+
ERC_D_Ln9.ne3pg3_ne3pg3_mt232.FHISTC_LTso.izumi_nag.cam-outfrq9s_nochem (Overall: DIFF) details:
71+
- Answer changes expected due to moving mountain bug fix
72+
73+
74+
izumi/gnu/aux_cam:
75+
ERC_D_Ln9.mpasa480_mpasa480_mt232.FHISTC_LTso.izumi_gnu.cam-outfrq9s_mpasa480 (Overall: DIFF) details:
76+
ERC_D_Ln9.ne3pg3_ne3pg3_mt232.FHISTC_LTso.izumi_gnu.cam-sat_lcltod (Overall: DIFF) details:
77+
ERC_D_Ln9.ne3pg3_ne3pg3_mt232.QPC7.izumi_gnu.cam-outfrq9s (Overall: DIFF) details:
78+
- Answer changes expected due to moving mountain bug fix
79+
80+
Summarize any changes to answers, i.e.,
81+
- what code configurations: All which use moving mountain gravity wave
82+
- what platforms/compilers: all
83+
- nature of change (roundoff; larger than roundoff but same climate; new
84+
climate): larger than roundoff, but same climate
85+
86+
If bitwise differences were observed, how did you show they were no worse
87+
than roundoff? Julio made a long run using these mods he put in cam6_4_126.
88+
89+
===============================================================
190
===============================================================
291

392
Tag name: cam6_4_135

0 commit comments

Comments
 (0)