-
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
Fixed issue with convert_culled_mesh_to_cdf5 for hurricane test case #726
Conversation
For hurricane test case
@gcapodag, this isn't passing pre-commit. Could you change the top of the from compass.ocean.tests.global_ocean.metadata import (
get_author_and_email_from_git,
) |
Flake8 is complaining about this: |
Okay, linting is good now. |
I don't think you should be using |
I didn't know, thanks @xylar |
This PR fixes this error:
when running the cull mesh step for the
mesh_lts
hurricane test case.Checklist
api.rst
) has any new or modified class, method and/or functions listedE3SM-Project
submodule has been updated with relevant E3SM changesMALI-Dev
submodule has been updated with relevant MALI changesTesting
in this PR) any testing that was used to verify the changescloses #725