Skip to content

Commit 445f78d

Browse files
authored
Merge pull request #4085 from ekluzek/merge-master-20260609
Merge ctsm5.4.044 into b4b-dev
2 parents 482e656 + 0702179 commit 445f78d

8 files changed

Lines changed: 401 additions & 31 deletions

bld/CLMBuildNamelist.pm

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2264,7 +2264,6 @@ sub setup_logic_params_file {
22642264
# get param data
22652265
my ($opts, $nl_flags, $definition, $defaults, $nl) = @_;
22662266

2267-
$nl_flags->{'use_hillslope'} = $nl->get_value('use_hillslope');
22682267
add_default($opts, $nl_flags->{'inputdata_rootdir'}, $definition, $defaults, $nl, 'paramfile',
22692268
'phys'=>$nl_flags->{'phys'},
22702269
'use_hillslope'=>$nl_flags->{'use_hillslope'},
@@ -3678,8 +3677,11 @@ sub setup_logic_hillslope {
36783677
add_default($opts, $nl_flags->{'inputdata_rootdir'}, $definition, $defaults, $nl, 'hillslope_transmissivity_method' );
36793678
add_default($opts, $nl_flags->{'inputdata_rootdir'}, $definition, $defaults, $nl, 'hillslope_pft_distribution_method' );
36803679
add_default($opts, $nl_flags->{'inputdata_rootdir'}, $definition, $defaults, $nl, 'hillslope_soil_profile_method' );
3680+
3681+
$nl_flags->{'use_hillslope'} = $nl->get_value('use_hillslope');
36813682
add_default($opts, $nl_flags->{'inputdata_rootdir'}, $definition, $defaults, $nl, 'use_hillslope_routing', 'use_hillslope'=>$nl_flags->{'use_hillslope'} );
36823683
add_default($opts, $nl_flags->{'inputdata_rootdir'}, $definition, $defaults, $nl, 'hillslope_fsat_equals_zero', 'use_hillslope'=>$nl_flags->{'use_hillslope'} );
3684+
36833685
my $use_hillslope = $nl->get_value('use_hillslope');
36843686
my $use_hillslope_routing = $nl->get_value('use_hillslope_routing');
36853687
if ( (! &value_is_true($use_hillslope)) && &value_is_true($use_hillslope_routing) ) {

doc/ChangeLog

Lines changed: 284 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,288 @@
11
===============================================================
2+
Tag name: ctsm5.4.044
3+
Originator(s): erik (Erik Kluzek,UCAR/TSS,303-497-1326)
4+
Date: Mon Jun 8 03:09:27 PM MDT 2026
5+
One-line Summary: Merge b4b-dev to master
6+
7+
Purpose and description of changes
8+
----------------------------------
9+
10+
Bring latest b4b-dev to master.
11+
12+
Mostly updates to the documentation, especially the tech note.
13+
14+
Also update of submodules to almost the latest ones. This includes a few updates in cdeps to help with spinup. One specific update is to allow using the CO2 from the CPLHIST files for spinup cases. Also use daily files for CO2 CPLHIST rather than 3-hourly to sync with the change in CMEPS.
15+
16+
Also update the FATES parameter generation for testing
17+
18+
Bugs fixed
19+
----------
20+
21+
List of CTSM issues fixed (include CTSM Issue # and description) [one per line]:
22+
- Resolves Update submodules to cesm3_0_alpha09c levels #4066 update submodules
23+
- Resolves Column level CH4 output not writing to history #4051 column level CH4 not working to history
24+
- Resolves Remove FatesSetupParamBuild #3989 Remove FatesSetupParamBuild
25+
- Resolves $4065 remove lists of tables/figs
26+
- Resolves Review 2.24. Plant Mortality #3870 plant mortality
27+
- Resolves Update documentation for RRTMGP fix related to sa_leaf (https://github.com/ESCOMP/CTSM/pull/3643) #3723 RRTMGP fix
28+
- Improves Review 2.31. Dust Model #3877 dust
29+
- Resolves Review 2.14. Model for Scale Adaptive River Transport (MOSART) #3860 MOSART
30+
- Resolved Review 2.9. Stomatal Resistance and Photosynthesis #3855 Stomatal resistance/Photosynthesis
31+
- Resolves Review 2.4. Radiative Fluxes #3850 Rad fluxes
32+
- Resolves Review 2.3. Surface Albedos #3849 surface albedos
33+
- Resolves Review 2.8. Snow Hydrology #3854 Snow hydrology
34+
- Resolves Review 2.25. Fire; then update #3871 fire
35+
- Resolves Fix numbering for numerical solution of vegetation temperature/fluxes in technical note section 2.5.3.2 #4025 numbering
36+
- Resolves Review 2.12. Lake Model #3858 Lake
37+
- Resolves Review 2.18. Plant Respiration #3864 plant respiration
38+
- Resolves User's Guide table missing: Required Files for Different Configurations and Simulation Types #2224 UG required files
39+
- Resolves User's Guide: Document FATES-CN options carbon_only / carbon_nitrogen #3957 UG Fates-CN options
40+
- Resolves User's Guide update: Section 1.5.7 BgcCrop spin-ups #3975 BGC spinup
41+
42+
Notes of particular relevance for users
43+
---------------------------------------
44+
Changes to CTSM's user interface (e.g., new/renamed XML or namelist variables):
45+
Add cplhist option to DATM_CO2_TSERIES XML option
46+
47+
Changes to documentation:
48+
Many updates to the Tech Note
49+
Some infrastructure updates
50+
Some updates to the User's Guide
51+
52+
Contributors:
53+
@slevis-lmwg, @samsrabin @sy-li @nmizukami @olyson @dmleung @ekluzek @swensosc @katyarjay @mvdebolskiy @cenlinhe @adrifoster
54+
55+
Notes of particular relevance for developers:
56+
---------------------------------------------
57+
Caveats for developers (e.g., code that is duplicated that requires double maintenance):
58+
Good news! The cime update from Sam R. fixes the permision problem we've been having on Izumi!
59+
60+
Changes to tests or testing:
61+
Remove FatesSetupParamBuild from testmods and includes as no longer needed with the JSON update
62+
Use COMP_ROOT_DIR_LND rather than SRCROOT so can be run from a CESM/CAM checkout
63+
64+
Testing summary: regular
65+
----------------
66+
[PASS means all tests PASS; OK means tests PASS other than expected fails.]
67+
68+
build-namelist tests (if CLMBuildNamelist.pm has changed):
69+
70+
derecho - OK
71+
72+
python testing (if python code has changed; see instructions in python/README.md; document testing done):
73+
74+
derecho - PASS
75+
76+
regular tests (aux_clm: https://github.com/ESCOMP/CTSM/wiki/System-Testing-Guide#pre-merge-system-testing):
77+
78+
derecho ----- OK
79+
izumi ------- OK
80+
81+
If the tag used for baseline comparisons was NOT the previous tag, note that here:
82+
83+
Answer changes
84+
--------------
85+
86+
Changes answers relative to baseline: No bit-for-bit
87+
88+
Other details
89+
-------------
90+
91+
List any git submodules updated (cime, rtm, mosart, cism, fates, etc.): cime, cmeps, cdeps. share, doc-builder
92+
cime to cime6.2.2
93+
cmeps to cmeps1.1.47
94+
cdeps to cdeps1.0.96
95+
share to share1.1.20
96+
doc-builder to v3.2.1
97+
98+
Pull Requests that document the changes (include PR ids): Total of 22
99+
(https://github.com/ESCOMP/ctsm/pull)
100+
101+
Changes to code: 4
102+
103+
- matlab script for appending WIEMIP co2 scenario data to TRENDY2025 #4082 matlab script
104+
- Update submodules resolving #4066 #4067 submodule updates
105+
- Make ch4 history accessible by field via the existing fincl approach #4055 ch4 history output
106+
- Fix fates paramgen in tests #4024 fix fates paramgen in tests
107+
108+
Changes to documentation: 18
109+
110+
Changes to Tech Note: 14
111+
112+
- Tech Note: Delete lists of figures and tables #4080 Delete lists of figures and tables
113+
- Update Plant Mortality tech note #4075 Plant Mortality
114+
- Add tree, shrub, and RRTMGP limitations on sa_stem and sa_leaf #4074 Tree/shrub limitations from the RRTMG change
115+
- Dust tech note update/typo fix #4073 Dust
116+
- add uuc equation #4057 add uuc equation
117+
- Update to tech note section 2-14 MOSART #4054 MOSART
118+
- Updates to Technote section 2.9 Stomatal Resistance and Photosynthesis #4053 Stomatal resistancea and photosynthesis
119+
- Updates to Technote section 2.4 Radiative Fluxes #4052 Rad fluxes
120+
- Update Technote for SNICAR snow albedo in Section 2.3 #4049 SNICAR
121+
- review Snow Hydrology Sect 2.8 in Technote #4048 Snow hydrology
122+
- Updates to CLM50_Tech_Note_Fire.rst by Fang Li #4043 Fire
123+
- Fix numbering sequence in technical note section 2.5.3.2 #4026 Fix equation numbering
124+
- Revision of section 2.12 (Lake Model) in technical note for CLM6 #3997 Lake model
125+
- 2.18 Plant Respiration Tech Note edits #3959 Plant respiration
126+
127+
Other documentation updates: 4
128+
129+
- Update doc-builder to v3.2 #4071 Update doc-builder
130+
- Remove section 1.4.3.1 What are the required files? #4050 Users' Guide
131+
- Update IMPORTANT_NOTES.md regarding fates_parteh_mode #4047 Just to IMPORTANT_NOTES
132+
- b4b-dev: Update bgc spinup section #3998 BGC spinup
133+
134+
===============================================================
135+
===============================================================
136+
Tag name: ctsm5.4.043
137+
Originator(s): wwieder (Will Wieder, UCAR/TSS)
138+
Date: Wed Jun 3 04:59:46 PM MDT 2026
139+
One-line Summary: Overflow respiration bug fixes
140+
141+
Purpose and description of changes
142+
----------------------------------
143+
144+
Same as title.
145+
146+
Significant changes to scientifically-supported configurations
147+
--------------------------------------------------------------
148+
149+
Does this tag change answers significantly for any of the following physics configurations?
150+
(Details of any changes will be given in the "Answer changes" section below.)
151+
152+
[Put an [X] in the box for any configuration with significant answer changes.]
153+
154+
[X] clm6_0 MIMICS only
155+
156+
[ ] clm5_0
157+
158+
[ ] ctsm5_0-nwp
159+
160+
[ ] clm4_5
161+
162+
163+
Bugs fixed
164+
----------
165+
166+
List of CTSM issues fixed (include CTSM Issue # and description) [one per line]:
167+
Resolves #3491
168+
169+
Notes of particular relevance for users
170+
---------------------------------------
171+
172+
Substantial timing or memory changes:
173+
A new failure, possibly a fluke, I opened an issue about it regardless:
174+
FAIL ERP_P64x2_D_Ld5.f10_f10_mg37.I2000Clm50Sp.derecho_gnu.clm-default--clm-nofireemis MEMLEAK memleak detected, memory went from 1039.200000 to 1168.030000 in 0 days
175+
176+
Contributors:
177+
@katierocci
178+
@slevis-lmwg
179+
180+
Notes of particular relevance for developers:
181+
---------------------------------------------
182+
183+
Changes to tests or testing:
184+
See note in memory changes above.
185+
186+
Testing summary:
187+
----------------
188+
189+
[PASS means all tests PASS; OK means tests PASS other than expected fails.]
190+
191+
regular tests (aux_clm: https://github.com/ESCOMP/CTSM/wiki/System-Testing-Guide#pre-merge-system-testing):
192+
193+
derecho ----- OK
194+
izumi ------- OK
195+
196+
Answer changes
197+
--------------
198+
Changes answers relative to baseline: Yes
199+
200+
Summarize any changes to answers, i.e.,
201+
- what code configurations: MIMICS only
202+
- what platforms/compilers: all
203+
- nature of change: larger than roundoff, reduces N limitation in MIMICS simulations
204+
205+
Other details
206+
-------------
207+
Pull Requests that document the changes (include PR ids):
208+
https://github.com/ESCOMP/ctsm/pull/4014
209+
210+
===============================================================
211+
===============================================================
212+
Tag name: ctsm5.4.042
213+
Originator(s): slevis (Samuel Levis,UCAR/TSS,303-665-1310)
214+
Date: Fri May 22 04:07:30 PM MDT 2026
215+
One-line Summary: Get hillslope_fsat_equals_zero .true. for use_hillslope
216+
217+
Purpose and description of changes
218+
----------------------------------
219+
220+
hillslope_fsat_equals_zero was intended to be .true. for use_hillslope = .true. but was coming back false:
221+
- First because of the lack of this line in CLMBuildNamelist.pm:
222+
$nl_flags->{'use_hillslope'} = $nl->get_value('use_hillslope');
223+
- And since ctsm5.4.040 due to the placement of said line in sub setup_logic_params_file, which is called after sub setup_logic_hillslope
224+
225+
Here I'm moving that line into sub setup_logic_hillslope, which fixes the problem without breaking anything else.
226+
227+
Significant changes to scientifically-supported configurations
228+
--------------------------------------------------------------
229+
230+
Does this tag change answers significantly for any of the following physics configurations?
231+
(Details of any changes will be given in the "Answer changes" section below.)
232+
233+
[Put an [X] in the box for any configuration with significant answer changes.]
234+
235+
[x] clm6_0 hillslope hydrology
236+
237+
[ ] clm5_0
238+
239+
[ ] ctsm5_0-nwp
240+
241+
[ ] clm4_5
242+
243+
244+
Bugs fixed
245+
----------
246+
List of CTSM issues fixed (include CTSM Issue # and description) [one per line]:
247+
Resolves #4030 Bug: hillslope_fsat_equals_zero is .false. when use_hillslope = .true.
248+
249+
Notes of particular relevance for users
250+
---------------------------------------
251+
Changes made to namelist defaults (e.g., changed parameter values):
252+
Now get the correct hillslope_fsat_equals_zero for use_hillslope = .true..
253+
254+
Testing summary:
255+
----------------
256+
257+
[PASS means all tests PASS; OK means tests PASS other than expected fails.]
258+
259+
build-namelist tests (if CLMBuildNamelist.pm has changed):
260+
261+
derecho - OK (2 expected failures)
262+
263+
regular tests (aux_clm: https://github.com/ESCOMP/CTSM/wiki/System-Testing-Guide#pre-merge-system-testing):
264+
265+
derecho ----- OK
266+
izumi ------- OK
267+
268+
Answer changes
269+
--------------
270+
Changes answers relative to baseline: Yes
271+
272+
Summarize any changes to answers, i.e.,
273+
- what code configurations: use_hillslope = .true.
274+
- what platforms/compilers: all
275+
- nature of change: larger than roundoff
276+
277+
No simulations have been run with the bug-fix, yet.
278+
279+
Other details
280+
-------------
281+
Pull Requests that document the changes (include PR ids):
282+
https://github.com/ESCOMP/ctsm/pull/4046
283+
284+
===============================================================
285+
===============================================================
2286
Tag name: ctsm5.4.041
3287
Originator(s): samrabin (Sam Rabin, UCAR/TSS)
4288
Date: Thu May 21 14:33:36 MDT 2026

doc/ChangeSum

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
Tag Who Date Summary
22
============================================================================================================================
3+
ctsm5.4.044 erik 06/08/2026 Merge b4b-dev to master
4+
ctsm5.4.043 slevis 06/03/2026 Overflow respiration bug fixes
5+
ctsm5.4.042 slevis 05/22/2026 Get hillslope_fsat_equals_zero .true. for use_hillslope
36
ctsm5.4.041 samrabin 05/21/2026 Merge b4b-dev 2026-05-21
47
ctsm5.4.040 multiple 05/20/2026 Paramfile updates
58
ctsm5.4.039 multiple 05/14/2026 Add FATES namelist option to initialize cohorts with diameter at breast height (DBH)

doc/README.CHECKLIST.master_tags.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ https://github.com/ESCOMP/ctsm/wiki/CTSM-development-workflow
2727
3c -- make sure you understand any changes to the baselines -- to document in ChangeLog
2828
3d -- Check the log file for run_sys_tests (`../run_sys_test.log`, to make sure that
2929
submodules are correct (see 2c above)
30-
3e -- When Izumi's baseline is ready, manually open read permissions to all.
3130
> [!TIP]
3231
> Always test on your fork with a feature-branch so that we can change tag order if needed. Put
3332
> baselines in the next tag name, as we can easily change afterwards if needed.
@@ -52,7 +51,7 @@ https://github.com/ESCOMP/ctsm/wiki/CTSM-development-workflow
5251
5e -- Push all the changes on your local branches to the branch on your fork
5352

5453
- [ ] 6. Submit a pull request (PR) for the changes
55-
Have someone review it if you are able. At minimum review it youself. The PR mechanism
54+
Have someone review it if you are able. At minimum review it yourself. The PR mechanism
5655
on git is an excellent way to code review code for both yourself and others. Also make
5756
sure all your changes are correct, changes that shouldn't have gone in don't, and all new
5857
files are added in.

0 commit comments

Comments
 (0)