Update bgc spinup section#3998
Open
slevis-lmwg wants to merge 23 commits into
Open
Conversation
slevis-lmwg
added
b4b
slevis-lmwg
added this to
CTSM: Upcoming tags, CLM documentation and LMWG: Sprint Planning Board
github-project-automation
Bot
moved this to Ready to start (or start again)
in CTSM: Upcoming tags
slevis-lmwg
moved this from Ready to start (or start again)
to In progress - b4b-dev
in CTSM: Upcoming tags
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Contributor
Author
|
@olyson could you review the spinup section that comes right after yours? I tried to simplify this section relative to the clm5 version since now there's the extra step of SASU. |
8 tasks
slevis-lmwg
commented
May 14, 2026
olyson
requested changes
May 14, 2026
Contributor
olyson
left a comment
olyson
left a comment
There was a problem hiding this comment.
Looks pretty good, thanks. Some comments on things that were probably there before regarding the case workflow that need fixing. I'm doing short runs of the sequence to make sure everything works, but submitting this review now so you can take a look.
github-project-automation
Bot
moved this from In progress - b4b-dev
to Stalled (needs review, blocked etc.)
in CTSM: Upcoming tags
…iogeochemistry-BGC-spinup.rst Add double quotes around module load ncl Co-authored-by: Keith Oleson <oleson@ucar.edu>
…gcspinups_sec1.5.7
This comment was marked as resolved.
This comment was marked as resolved.
Contributor
Author
|
Thank you @olyson, and I'm sorry that you discovered so many unaddressed issues. I have included your recommendations (except for changing Sp to SP, though I'm open to discussing it), and I will take responsibility for rerunning the examples to confirm that they work with my updates. |
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
2 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
Start with the issue #3975
Specific notes
Contributors other than yourself, if any:
CTSM Issues Fixed (include github issue #):
Resolves #3975
Are answers expected to change (and if so in what way)? No
Any User Interface Changes (namelist or namelist defaults changes)? No
Does this create a need to change or add documentation? Did you do so?
This PR updates documentation.
/glade/work/slevis/git/ans_chging_tags/tools/contrib. Use them to replace all 'placeholder.png' in my .rst file mods.