-
Notifications
You must be signed in to change notification settings - Fork 37
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 util for culling MPAS-Ocean and -Seaice restart files #794
Conversation
TestingI ran this in:
using the following restart files:
|
9647f27
to
08589d6
Compare
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.
I didn't run this myself, but the code changes and the restart files you linked to look good by visual inspection. I am comfortable with you merging this when you like and we can push fixes if any issues are revealed in the upcoming IcosX runs. Thanks for your work on this!
@cbegeman, I agree. Let's get this merged and we can tweak if necessary. |
d2eb7f0
to
3b8952d
Compare
@cbegeman thanks so much for the review! |
This needs a new version of MPAS-Tools (not yet released) with MPAS-Dev/MPAS-Tools#557
Checklist
api.rst
) has any new or modified class, method and/or functions listedTesting
in this PR) any testing that was used to verify the changes