-
Notifications
You must be signed in to change notification settings - Fork 313
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move plumber2 scripts to python directory #2503
Conversation
These failures are already tracked with derecho versions
This will be reverted once issue ESCOMP#2423 has been addressed
This also adds a corresponding namelist build unit test and similar test to check that sp mode + spitfire fails
Merge b4b-dev Some fixes for run_neon that came up in ctsm5.1.dev172. Small bug fix for units of vector format hillslope hydrology. Start adding PLUMBER2 users-mods (NOT functional)
FATES API35 parameter file update
Here is the probably unrelated error in testing that I mentioned above:
|
@TeaganKing thanks for working on bringing this in. A small formatting suggestion for PR's and issues. When you have test results that come in, it'll format it weirdly naturally. But, if you highlight and make it code with the "<>" option above the comment it'll look more normal. |
Thanks @ekluzek . I had pressed '<>' before adding the code, and it's certainly not a one-line bit of code, so the single quotes didn't work as I intended-- I fixed that though :) I think I also must have not set up merging into b4b dev correctly, as this is also pulling in changes from you and Gregory? I'm assuming these changes should go into b4b dev before master? |
@TeaganKing I'm thinking you just need to merge in the latest escomp/b4b-dev. Fetch and then do that and it should clean that up. If not we'll need to figure that out. One other suggestion is that you make sure the issues fixed are on it's own line with a "fixed" in front of it. That will it will know to autoclose the issue. This is something we should make more obvious in the PR template. |
This is now being done in #2505 so that it is off of the correct branch... closing this PR. |
Description of changes
This PR addresses @2187 by moving new scripts that were added in #2155 to tools/site_and_regional into the python directory, similarly to other NEON scripts in #2156.
Fixed: #2187
Are answers expected to change (and if so in what way)? No
Any User Interface Changes (namelist or namelist defaults changes)? No
Testing performed, if any:
I tested newly added unit test, which was successful, with
python -m unittest test_unit_plumber2_surf_wrapper.py
. I will add that when testing with./run_ctsm_py_tests -u
, I ran into a seemingly unrelated error that I'll add in a comment below.