Skip to content

Conversation

djk2120
Copy link
Contributor

@djk2120 djk2120 commented Aug 28, 2025

Description of changes

Adding a new tool to tether CTSM simulations automatically.

Specific notes

Contributors other than yourself, if any:
Advice and guinea pig feedback from @linniahawkins @samsrabin @adrifoster

CTSM Issues Fixed:

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

Any User Interface Changes (namelist or namelist defaults changes)? This would create a new tool for automating submission of tethered CTSM simulations, but otherwise doesn't actually change the basic CTSM user interface.

Does this create a need to change or add documentation? Did you do so? Yes, this will need documentation. I've started drafting documentation in tools/tether/README.md. It's likely that we'll want to have generic "tether" documentation, as well as specific documentation for any supported sequences, e.g. SASU spinup, TRENDY S3, etc.

Testing performed, if any: None

@samsrabin samsrabin added enhancement new capability or improved behavior of existing capability bfb bit-for-bit labels Aug 28, 2025
@samsrabin
Copy link
Member

Great to see this, Dan, thanks! I'm going to unsubscribe for now to reduce my notification load, but please ping me with any questions or if you want me to have a look.

@ekluzek
Copy link
Collaborator

ekluzek commented Aug 29, 2025

Cool, thanks for making this a PR @djk2120. It's helpful to start to look through the code. I can tell that I'll need you to do a run-through of the code with us at some point.

We should probably rebase this to b4b-dev, to come in sooner. But, we'll wait for you to feel it's closer to ready.

@djk2120
Copy link
Contributor Author

djk2120 commented Sep 8, 2025

OK, I have a first draft in place. The tether utility and a spinup stability script are in place. And then two main examples ("example1" and "sasu_spinup") for how they could be deployed. What are the next steps @samsrabin @ekluzek ? Should I present the code at an upcoming software engineering meeting (or a smaller meeting)? Does it make sense for somebody else to beta-test one or both of the examples?

@samsrabin samsrabin added the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Sep 8, 2025
@samsrabin
Copy link
Member

samsrabin commented Sep 8, 2025

Excellent, thanks! Before one of us starts testing, could you merge in the ctsm5.3.073 tag and make sure it still works for you?

git remote add escomp git@github.com:ESCOMP/CTSM.git
git fetch escomp
git merge --no-ff ctsm5.3.073

@samsrabin
Copy link
Member

(Or alternatively, rebase onto that tag if that's something you're comfortable doing.)

Update .gitmodules to cesm3_0_alpha07c

 Update ccs_config_cesm1.0.48 to ccs_config_cesm1.0.56.
 Update cime6.1.112 to cime6.1.113.
 Answers change in gnu and nvhpc tests on derecho (details in the PR).
 New bugs found and reported in issues
 ESCOMP#3453 FAIL MKSURFDATAESMF_...intel NLCOMP
 ESCOMP#3454 FAIL SUBSETDATA* tests NLCOMP

 PR ESCOMP#3422
Preparing beta version of tether utility
@djk2120
Copy link
Contributor Author

djk2120 commented Sep 10, 2025

OK, @samsrabin. I was able to merge in the new tag uneventfully.

@samsrabin
Copy link
Member

Nice, thanks! I've marked this as "next" so we should discuss it in our next SE meeting.

@djk2120
Copy link
Contributor Author

djk2120 commented Sep 12, 2025

I'm also realizing that maybe this is actually two tools: a spinup stability python tool and a tethering utility, which I should probably split into two folders in tools/ (and potentially split into two PRs?)

@ekluzek ekluzek requested a review from wwieder September 12, 2025 20:23
@ekluzek ekluzek removed the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Sep 12, 2025
@samsrabin
Copy link
Member

@wwieder will test this in some upcoming spinup runs. @adrifoster will also test. @djk2120 let's keep this as one PR, but it'd be good if you could go ahead and split out the code for the spinup stability script.

@samsrabin samsrabin changed the title Tether (DRAFT PR) Tethering Sep 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bfb bit-for-bit enhancement new capability or improved behavior of existing capability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

translate spinup stability script to python update infrastructure to support tethered simulations
5 participants