Update COMPASS scripts to support python 3#177
Update COMPASS scripts to support python 3#177mark-petersen merged 5 commits intoMPAS-Dev:developfrom
Conversation
This isn't related to python 3, just something that drives me nuts about the script...
Also, remove an unneeded import
|
@mark-petersen, could you make sure these changes work for you when using COMPASS as normal? If so, we'll want to merge |
|
@matthewhoffman, could you make sure land-ice test cases still work as expected with these updated versions of the scripts? |
|
The intention is just to test that these don't break python 2.7 support for now. Many other COMPASS scripts aren't ready for python 3 yet, so that testing will need to wait. |
mark-petersen
left a comment
mark-petersen
left a comment
There was a problem hiding this comment.
Tested all scripts:
clean_testcase.py list_testcases.py manage_regression_suite.py setup_testcase.py
for --help, case listing, case creation, ran a single setup case, ran a nightly regression, and cleaned a case. I committed a few small updates, and everything works.
|
Asked @matthewhoffman and he was satisfied with my ocean tests. Removing him as a reviewer. |
|
@mark-petersen, sorry about that. Thanks for the fixes. Any reason I shouldn't go ahead and merge this to |
|
Okay, glad I asked. I'll leave it to you. |
The changes mostly involve fixing print statements. The list command has also been modified to produce a sorted list of test cases (as opposed to the random order that was previously produced).
The utility_scripts were not included in #177 Fixes a bug by decoding the git_version into a unicode string in manage_regression_test_suite.py, required for python 3 compatibility.
…r to update global attributes (MPAS-Dev#177) * add a new stream property 'gattr_update' to control whether to update global attributes * fix a typo: interger -> integer * bump version to 8.3.1-2.5
3 participants
The changes mostly involve fixing
printstatements.The list command has also been modified to produce a sorted list of test cases (as opposed to the random order that was previously produced).