-
Notifications
You must be signed in to change notification settings - Fork 176
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
Add echgres as a dependency for earc #3202
Conversation
@CatherineThomas-NOAA Can you please incorporate this change in the C1152 test that discovered this bug. The current CI testing will likely not run into this issue. |
@aerorahul: I'll pull this change into gfs_tasks.py, regenerate the xml, and continue the experiment. |
The XML was generated successfully. The difference:
I'll let the experiment get through one earc step before approving. |
looks correct to me. |
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.
My C1152 exp cycled over the weekend with no earc issues. There was even a case where echgres finished after epos, and earc properly waited until after echgres completed before beginning. Approve.
Description
This PR:
echgres
as a dependency to theearc
job.Resolves Ensemble archive task dependency needs to include echgres #3201
Resolves Missing enkfgdas_echgres dependency for metatask eanm #3165
Type of change
Change characteristics
How has this been tested?
Checklist