Teagan King

Ok, thank you both for this feedback!

That sounds good. I have successfully run the AD & postAD case (with a bit of hard-coding), so this will require the following updates, which should be feasible: - [x]...

Ok, in that case, I'll just plan to change the default for PLUMBER to AD and add some documentation on this anticipated workflow for users. Thanks for clarifying! Why don't...

This is about ready to go, but once #2485 is in, I'll run the unit and system tests once more and perform a few final tests to make sure the...

I just wanted to note here a few things that I still need to fix: - [x] Tests for `test_sys_run_tower.py` are currently failing. Fix and then run all ctsm py...

One limitation of this implementation is that if both neon and plumber cases are run simultaneously (not recommended), that the default run type becomes transient. While this could be changed...

Tests on derecho currently pass for `make all` and `./run_ctsm_py_tests --sys`. The `test_sys_run_tower.py` tests take about 30 minutes to run now that it involves setting up three cases; I can...

Hi @ekluzek , I have now addressed the comments from @wwieder 's review. Two outstanding issues that Will and I discussed are the following (more details in comments above): -...

Per conversation with Erik, we can remove the [files listed in PLUMBER2 user mod directories](https://github.com/ESCOMP/CTSM/blob/master/cime_config/usermods_dirs/PLUMBER2/defaults/user_nl_datm_streams ) because these will be implemented in another PR (#277 ); those do not need...

The `dtlimit` is updated as expected when running CTSM when I do an `xmlchange` to set `CLM_USRDAT_NAME` to `PLUMBER`. So, @slevis-lmwg , I think we can run the aux_cdeps test-suite....