Teagan King
Teagan King
> Sorry yes, [this one that was already merged?](https://github.com/ESCOMP/CTSM/pull/2155) Oh, I see it now! Thanks!
I also did uncomment the following line in the last commit given that this will come in after the CDEPS PR: `./xmlchange PLUMBER2SITE='+site + '\n'` This should now be ready...
The merge conflicts are fixed now, too. Some variables had already been renamed in b4b-dev to conform to snake case, and the plumber2_usermods.py file had been moved to the python...
Ready for review with the caveat of the dependency of CDEPS PR 262 (as the label on this PR suggests). I can also leave this as not ready for review...
- [x] Last dependency is 262 . Adding this unchecked task to make sure this doesn't get merged in accidentally.
When running PLUMBER with this 5.2 dataset, it looks like there is some missing boundary data for PCT_OCEAN (see below). @slevis-lmwg or @wwieder , do we need to provide or...
Thanks @ekluzek !
[CDEPS #262](https://github.com/ESCOMP/CDEPS/pull/262) is basically done, but just needs the merge button to be clicked. I was planning on adding that CDEPS tag to #2406 ; the two questions I have...
> Does this one require the CDEPS tag for it to function? It looks like it does to me -- but I could be wrong. If it does I'd include...
Ok, thanks @ekluzek for your help! The `cdeps` tag has been updated and this is ready for review!