Update crops/irrigation Tech Note chapter for 6.0#4058
Draft
samsrabin wants to merge 87 commits into
Draft
Task list completed / task-list-completed
Started
2026-06-19 18:11:30
ago
55 / 77 tasks completed
22 tasks still to be completed
Details
Required Tasks
| Task | Status |
|---|---|
| Local preview | Incomplete |
| I have followed the CTSM contribution guidelines. | Completed |
| Extract docs infrastructure commits into their own PR; merge; rebase this onto that. | Incomplete |
| "Summary of CLM5.0 updates relative to the CLM4.5", including "Available new features since the CLM5 release" | Completed |
| "Separate reproductive pool"? Seems redundant with information already elsewhere. | Incomplete |
| References to recent papers? | Incomplete |
| References to CLM5, CLM5-BGC-crop, etc.: Convert to just CLM or CTSM | Completed |
| No more "the CLM" | Completed |
| Remove references to Agro-IBIS, except at beginning. (Maybe?) | Incomplete |
| Use |
Completed |
Be more careful about hybgdd (parameter for starting point of calculating |
Completed |
|
|
Completed |
| Relabel the ones whose labels are just numbers (do this last, because later notes here refer to the number labels) | Incomplete |
Replace * with \times
|
Completed |
Be consistent about stem vs. livestem
|
Completed |
| Replace math with literal, where appropriate | Incomplete |
| Replace italics with math, where appropriate | Completed |
| Consistently use C_3 and C_4. | Completed |
| Introduction: Crops are much more common now in ESMs. Condense this section greatly to just introduce the idea of the crop model and its inheritance from AgroIBIS. | Completed |
| Link to docs on "land unit" and "soil column" | Completed |
| "Crop grid cell coverage ...": Just refer to relevant other sections of the documentation. | Completed |
| Delete: "The representations of sugarcane, rice, cotton, tropical corn, and tropical soy were new in CLM5; miscanthus and switchgrass were added after the CLM5 release." We don't want our tech note to be a history lesson. | Completed |
| What is "functional form"? | Completed |
| No longer accurate; delete? "In tropical regions, parameter values were developed for the Amazon Basin, and planting date window is shifted by six months relative to the Northern Hemisphere." | Completed |
| spatially? | Incomplete |
Describe how to get this information from the paramfile. Unfortunately query_paramfile doesn't make it easy yet: ESCOMP/CTSM Issue #3457: query_paramfile: Better presentation of mergetoclmpft
|
Completed |
| Mention that trying to enable the inactive crop types will NOT WORK because of ESCOMP/CTSM Issue #3388: Avoid in-code checks of PFT type | Completed |
| Add text about my reparameterization? Nothing there about CLM5 reparameterization... | Incomplete |
| Simplify crop PFT table | Completed |
| Delete bit at beginning describing non-crop phenology types | Completed |
| Delete: "(for the northern hemisphere)", "In the southern hemisphere (SH) the NH requirements apply 6 months later." | Completed |
| Mention new sowing windows derived from GGCMI; cite Rabin et al. (2023) | Completed |
| Reorganize. Maybe have this not be an equation at all? | Completed |
Edit to account for now using rx_cultivar_gdds_thisyr_patch, gdd20_baseline_patch, etc. |
Completed |
| Add bioenergy crops | Completed |
| Add winter wheat | Completed |
| Change label to something descriptive | Completed |
| Things like 1850 for corn/sugarcane are actually parameters! But 950 isn't. Fix all this. | Completed |
Account for min_gddmaturity
|
Completed |
If any \circs are left after all that, replace them with proper degree symbols |
Completed |
| Maximum increment needs to be divided by 48 (timesteps per day) | Completed |
| Leaf emergence section: Clarify that leaf emergence happens at BEGINNING of this phase, and that the phase as a whole might be better thought of as "vegetative growth." | Completed |
| Remove sowing window dates (rows and also following bullet point—or maybe keep bullet point to say "these variables are what you would have had before CLM5.x", or how you can simply set sowing date for single-site runs) | Completed |
| Add netCDF variable names | Completed |
| Replace math with literals, where it makes sense to do | Completed |
| Update values | Completed |
| Add disclaimer | Completed |
| See #4084 | Incomplete |
| "using the BGC leaf offset algorithm" ?? | Incomplete |
| "Allocation changes based on the crop phenology phases phenology" | Completed |
| Update, if needed, based on my reparameterization (although I don't think it should be): "All C:N ratios were determined by calibration process, through comparisons of model output versus observations of plant carbon throughout the growing season." Not needed, based on #3662 and #3686. | Completed |
"The BGC part of the model keeps track of a term representing excess maintenance respiration, which supplies the carbon required for maintenance respiration during periods of low photosynthesis (Chapter :numref:rst_Plant Respiration).": Delete this. Instead, just cross-ref the section in the next sentence. |
Completed |
| Second row: what about where that term is > 1? | Completed |
| Change label to something descriptive | Completed |
Clarify SOME lfemerg phase allocation to leaf when max LAI has been reached |
Completed |
| Rows 2 and 4: what about where that term is > 1? | Completed |
Those ≤ 1s can't be right. |
Completed |
| Change label to something descriptive | Completed |
Explain grainfill phase allocations when max LAI has been reached |
Completed |
"Nitrogen allocation for crops ... is supplied in CLM by the soil mineral nitrogen pool": What about BNF? Reference section Biological nitrogen fixation for soybeans here. |
Incomplete |
| "post-grain fill C:N ratios are assigned the same as crop residue": This can't be right... Can it? How is "crop residue" assigned a C:N ratio and not an emergent property? | Incomplete |
Explain how grain N works. Reference $\paramgraincn$, presumably. |
Incomplete |
It sounds like biofuel harvest goes to the _TO_FOOD history variable?? |
Completed |
| Update values | Completed |
| Add disclaimer | Completed |
| "based on the LUMIP land use and land cover change time series (LUH2 for historical and SSPs for future)": Update? | Incomplete |
| "although denitrification rates are slower in the current version of the model": cited paper is pretty old. Maybe not "the current version" anymore. | Incomplete |
| "Unlike natural vegetation, where a fraction of each PFT are N fixers, all soybeans are treated as N fixers." Well, no. That's a fraction too: 1. | Completed |
| Grain C equation: Extraneous parenthesis | Completed |
| Fix "tempereature" typo | Completed |
| How to turn this off? | Incomplete |
| Move this to be with other base temp / GDD calculation info | Incomplete |
| "includes the option", "When irrigation is enabled": It's actually on by default (MAYBE: WAITING ON #4056) | Incomplete |
| "$f_{thresh}$ is a tuning parameter": What value? Hard-coded? Variable name? | Incomplete |
| "To conserve mass" options: Which is on by default? How to change? | Incomplete |
| Check/update values | Incomplete |
Add parameter names; mention that these are set in user_nl_clm
|
Completed |
Loading