Skip to content

more information regarding PLUMBER 2 simulations using the ./runtower…#3305

Merged
slevis-lmwg merged 16 commits intoESCOMP:b4b-devfrom
ijaguirre:run_tower_more_documentation
Mar 18, 2026
Merged

more information regarding PLUMBER 2 simulations using the ./runtower…#3305
slevis-lmwg merged 16 commits intoESCOMP:b4b-devfrom
ijaguirre:run_tower_more_documentation

Conversation

@ijaguirre
Copy link

@ijaguirre ijaguirre commented Jul 6, 2025

Description of changes

More documentation for the PLUMBER 2 option using the newly developed ./run_tower tool created by @TeaganKing

Specific notes

Contributors other than yourself, if any:

CTSM Issues Fixed (include github issue #):
Related to #2997

Are answers expected to change (and if so in what way)?
No. Documentation only.

Any User Interface Changes (namelist or namelist defaults changes)? No

Does this create a need to change or add documentation? Did you do so? No

Testing performed, if any: None

@TeaganKing TeaganKing self-requested a review July 7, 2025 03:53
Copy link
Contributor

@TeaganKing TeaganKing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for putting this together, @ijaguirre !

I have a few clarifying questions for you, and am happy to address the other changes myself if they seem reasonable to you! If you could just answer a few of the questions and thumbs up the comments if you agree with them, or of course share other thoughts, I'm happy to wrap this up!

@ekluzek ekluzek added the documentation additions or edits to user-facing documentation or its infrastructure label Jul 7, 2025
@ekluzek
Copy link
Collaborator

ekluzek commented Jul 7, 2025

I'm rebasing the PR to come to b4b-dev. We may also need to rebase the branch itself, depending on where it's at.

@ekluzek ekluzek changed the base branch from master to b4b-dev July 7, 2025 17:13
@ekluzek ekluzek added next this should get some attention in the next week or two. Normally each Thursday SE meeting. test: docs Test documentation build before merging docs-loc:users-guide Relates to User's Guide (using CTSM/tools) labels Jul 7, 2025
@TeaganKing
Copy link
Contributor

I'm rebasing the PR to come to b4b-dev. We may also need to rebase the branch itself, depending on where it's at.

Thanks @ekluzek ! I think that the branch itself should be fine, at least with regards to this documentation file. I don't think the file has been otherwise updated recently.

@samsrabin samsrabin self-assigned this Jul 10, 2025
@ekluzek ekluzek removed the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Jul 10, 2025
@wwieder
Copy link
Contributor

wwieder commented Jan 5, 2026

@ijaguirre have you been able to make the changes / clarifications that @TeaganKing requested?

@ekluzek ekluzek added the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Jan 8, 2026
@ijaguirre
Copy link
Author

Dear @TeaganKing, I apologize for the delay with the corrections. I made all the changes you asked for. If you have any more comments, I'll make sure to update the notes within 1-2 days.

Warm regards,
Ignacio

@ijaguirre
Copy link
Author

ijaguirre commented Jan 19, 2026 via email

@ijaguirre
Copy link
Author

Dear all,

I added further information and an example.

@wwieder wwieder removed the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Jan 29, 2026
@wwieder wwieder requested a review from ekluzek January 29, 2026 17:27
@samsrabin samsrabin self-requested a review February 5, 2026 16:58
@samsrabin samsrabin moved this from Todo to In Progress in CLM documentation Feb 5, 2026
@slevis-lmwg
Copy link
Contributor

Thank you @ijaguirre
I will go through and mark all conversations "resolved"

@slevis-lmwg slevis-lmwg added PR status: ready PR: this is ready to merge in, with all tests satisfactory and reviews complete bfb bit-for-bit labels Mar 16, 2026
@slevis-lmwg slevis-lmwg removed the PR status: ready PR: this is ready to merge in, with all tests satisfactory and reviews complete label Mar 18, 2026
@slevis-lmwg
Copy link
Contributor

I found evidence of (some of) the errors by clicking "View details" and scrolling through the many lines:

/home/user/mounted_home/doc/source/users_guide/running-single-points/supported-tower-sites.rst:81: ERROR: Unexpected indentation. [docutils]
/home/user/mounted_home/doc/source/users_guide/running-single-points/supported-tower-sites.rst:92: WARNING: Enumerated list ends without a blank line; unexpected unindent. [docutils]
/home/user/mounted_home/doc/source/users_guide/running-single-points/supported-tower-sites.rst:94: ERROR: Unexpected indentation. [docutils]
/home/user/mounted_home/doc/source/users_guide/running-single-points/supported-tower-sites.rst:98: WARNING: Enumerated list ends without a blank line; unexpected unindent. [docutils]
/home/user/mounted_home/doc/source/users_guide/running-single-points/supported-tower-sites.rst:100: ERROR: Unexpected indentation. [docutils]

@ijaguirre could you look into resolving?

@ijaguirre
Copy link
Author

ijaguirre commented Mar 18, 2026 via email

@samsrabin
Copy link
Member

@ijaguirre Here is our User Guide section on working with the documentation. It includes some specific guidance on the "unexpected indentation" error, as well as a different "unexpected indent" error.

Copy link
Contributor

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay, I got it all to work after some trial and error...

@slevis-lmwg slevis-lmwg merged commit 42ab536 into ESCOMP:b4b-dev Mar 18, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in CLM documentation Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bfb bit-for-bit docs-loc:users-guide Relates to User's Guide (using CTSM/tools) documentation additions or edits to user-facing documentation or its infrastructure test: docs Test documentation build before merging

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants