First set of crop parameter changes for CLM6#3574
Closed
samsrabin wants to merge 1 commit into
Closed
Conversation
Calibrating yield for about 2000-2024 while minimizing failed seasons: - Corn: Don't change anything. It's high for 2000-2010 but looks good after that. - Cotton: Increase max growing season length from 160 days to 270 days. Causes overwintering, so this will probably need to be reverted! But useful for now to explore how much an extreme change could help with the high failure rate. - Rice: Revert CLM5 `arooti` changes. - Soybean: It's low for this period but nothing I tried helped much. Removing mxmat helped a bit, so increase max growing season length from 150 days to 180 days. - Sugarcane: Increase max growing season length from 300 days to 360. - Wheat: Revert CLM5 arooti changes. Need a pretty big drop in yields and this didn't do it in my test, but that was without grainfill triggering max LAI and thus much more failed crop area. (There were no crop phase parameter changes in CLM5.)
samsrabin
added
the
enhancement
samsrabin
added
science
github-project-automation
Bot
moved this to Status not assigned
in CTSM: release-ctsm5.4 minor version update (CMIP7 datasets)
62 tasks
Contributor
|
I'm pretty agnostic to when and where this gets brought in. Here are a few thoughts:
|
Member
Author
|
Talked to @ekluzek. I will bring in the alpha5.4 parameter file plus my changes onto master directly. |
ekluzek
moved this from Status not assigned
to In progress
in CTSM: release-ctsm5.4 minor version update (CMIP7 datasets)
Member
Author
|
Replaced by #3618. |
github-project-automation
Bot
moved this from In progress
to Done
in CTSM: release-ctsm5.4 minor version update (CMIP7 datasets)
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of changes
Calibrating yield for about 2000-2024 while minimizing failed seasons:
arootichanges.Specific notes
Contributors other than yourself, if any: None
CTSM Issues Fixed: None
Are answers expected to change (and if so in what way)? Yes, for any run with prognostic crops.
Any User Interface Changes (namelist or namelist defaults changes)?
paramfileforclm6_0.Does this create a need to change or add documentation? Did you do so? Yes; no.
Testing performed, if any: Scientific testing of the same changes on top of
master's paramfile looked good.aux_clm