-
Notifications
You must be signed in to change notification settings - Fork 320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New outputs: N to food and seed #2074
Merged
Merged
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
93e82dc
Add N_TO_FOOD outputs.
samsrabin 2cd8339
Add N_TO_SEED outputs.
samsrabin f737631
Make N_TO_FOOD and N_TO_SEED outputs default inactive.
samsrabin 8ae1ecb
Fix indentation of a block.
samsrabin bda8cf1
Merge branch 'master-upstream' into grainN-to-food
samsrabin 01731b7
Add N_TO_FOOD perharv and ann.
samsrabin a26a5d1
Merge tag 'ctsm5.1.dev142' into grainN-to-food
samsrabin bf9f4d8
Update history_fields .rst files.
samsrabin eeab40d
Add GRAINN_TO_FOOD to first history file in crop testmod.
samsrabin d0c44a1
Added TO_SEED _ANN and _PERHARV outputs.
samsrabin b95b986
Fix in-progress attempt to add note to N_TO_FOOD longnames.
samsrabin 205848b
Add "not scientifically supported" to reproductive N hist field longn…
samsrabin 6da9d40
Merge tag 'ctsm5.1.dev146' into grainN-to-food
samsrabin ae8f46a
Move GRAINC_TO_SEED_ANN to same place as GRAINC_TO_SEED.
samsrabin af4a688
Added *C_TO_SEED_PERHARV hist fields.
samsrabin b543f07
Merge tag 'ctsm5.1.dev147' into grainN-to-food
samsrabin 1e2c520
Updated history_fields*.rst files.
samsrabin 333c0d6
Update ChangeLog and ChangeSum.
samsrabin File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,49 @@ | ||
=============================================================== | ||
Tag name: ctsm5.1.dev148 | ||
Originator(s): samrabin (Sam Rabin, UCAR/TSS, [email protected]) | ||
Date: Fri Nov 3 07:56:43 MDT 2023 | ||
One-line Summary: Add GRAINN outputs | ||
|
||
Purpose and description of changes | ||
---------------------------------- | ||
|
||
In response to a user request for GRAINN_TO_FOOD outputs, this adds *_N_TO_FOOD(_ANN) and *_N_TO_SEED(_ANN) outputs for reproductive N pools. These are off by default, unlike their C counterparts. Note that the results are not scientifically supported, and tests have revealed unrealistic values. (Also adds GRAINC_TO_SEED_ANN output.) | ||
|
||
|
||
Significant changes to scientifically-supported configurations | ||
-------------------------------------------------------------- | ||
|
||
Does this tag change answers significantly for any of the following physics configurations? | ||
(Details of any changes will be given in the "Answer changes" section below.) | ||
|
||
[ ] clm5_1 | ||
|
||
[ ] clm5_0 | ||
|
||
[ ] ctsm5_0-nwp | ||
|
||
[ ] clm4_5 | ||
|
||
|
||
Testing summary: | ||
---------------- | ||
|
||
[PASS means all tests PASS; OK means tests PASS other than expected fails.] | ||
|
||
regular tests (aux_clm: https://github.com/ESCOMP/CTSM/wiki/System-Testing-Guide#pre-merge-system-testing): | ||
|
||
cheyenne ---- OK (with some fieldlist diffs) | ||
izumi ------- PASS (with some fieldlist diffs) | ||
|
||
|
||
Other details | ||
------------- | ||
|
||
Pull Requests that document the changes (include PR ids): | ||
* #2074 (https://github.com/ESCOMP/CTSM/pull/2074) | ||
|
||
=============================================================== | ||
=============================================================== | ||
Tag name: ctsm5.1.dev147 | ||
Originator(s): samrabin (Sam Rabin, UCAR/TSS, [email protected]) | ||
Date: Mon Oct 30 16:53:20 MDT 2023 | ||
|
This file contains 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
35 changes: 6 additions & 29 deletions
35
doc/source/users_guide/setting-up-and-running-a-case/history_fields_fates.rst
Large diffs are not rendered by default.
Oops, something went wrong.
81 changes: 75 additions & 6 deletions
81
doc/source/users_guide/setting-up-and-running-a-case/history_fields_nofates.rst
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I asked @samsrabin to remind me the diff between "grain to seed" and "grain to food" and I see here that the two terms summed are the reproductive pool. I'm not sure "seed" is the right term for what's left when you subtract food from the reproductive pool, but that's out of scope for this PR.