-
Notifications
You must be signed in to change notification settings - Fork 86
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
[develop] Add MET/METplus to Jet #799
[develop] Add MET/METplus to Jet #799
Conversation
Did you check the MET outputs and logs? I had hoped to put this change in #779 but ran into errors from MET, even though the workflow tasks succeeded. I was thinking it needs an appropriate run_vx.local modulefile and/or updates to the miniconda environment. |
Hi @willmayfield - I didn't check the logs or outputs. What sort of errors did you receive? I'm noticing the |
Yep, that's what I see too. |
Added |
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.
Compared a few MET stat files to the same test on Hera. There were very small numerical differences, likely within reason between platforms, and the MET logs looked fine. Thanks Ed!
Thanks @willmayfield for doing the comparison! |
DESCRIPTION OF CHANGES:
Added MetPlus file paths to the Jet machine file in the develop branch. A similar PR #797 was approved for the release branch.
TESTS CONDUCTED:
Ran
MET_verification
andMET_ensemble_verification
successfully on Jet.DEPENDENCIES:
None.
DOCUMENTATION:
No documentation changes are needed.
ISSUE (optional):
CONTRIBUTORS (optional):