-
Notifications
You must be signed in to change notification settings - Fork 32
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
Testing of the Marine DA jjobs #234
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few comments/questions
Automated Global-Workflow GDASApp Testing Results:
|
Looks like I hard-coded |
Automated Global-Workflow GDASApp Testing Results:
|
Automated Global-Workflow GDASApp Testing Results:
|
Automated Global-Workflow GDASApp Testing Results:
|
Automated Global-Workflow GDASApp Testing Results:
|
We shall see, but I think all our test should be re-runable (it's a word ... ) now. |
Automated Global-Workflow GDASApp Testing Results:
|
Automated Global-Workflow GDASApp Testing Results:
|
Re-requesting your review @CoryMartin-NOAA , I had to add a few changes to make some of the ctest "re-runable". |
Adds the UFSDA location to PYTHONPATH in soca prep jjob so the path building is not done in script. Refs: NOAA-EMC/GDASApp#242 and NOAA-EMC/GDASApp#234 The issues require changes also to GDASApp, but this PR will not break GDASApp as is
Still a bit of a work todo but good enough for a draft PR.
Addition of 3 ctests related to running with the g-w
setup_expt.py
test adjusted tosoca
Stuff left to do
- As I type this I'm wondering if the changes I made to the g-w are needed (probably not)Issues addressed