-
Notifications
You must be signed in to change notification settings - Fork 1
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
Basic regrid operator #405
Conversation
There is some repetition within both operators in determining the x,y coordinate names. We might want to have this as a separate operator/utility? |
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.
Overall looking like a solid addition to CSET capability, and one that will be very useful going forwards.
I've noted some things that need tweaking, along with some things I would like additional documentation for, and wanting some additional tests. I'm happy to discuss how to go about these various things.
tidy docstring Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
update copyright Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
update docstring Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
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 think there is a small logic error causing all supported cubes to be rejected.
Fixes a pre-commit issue apparently only in CI?
I'm not sure why the pre-commit check was not failing locally, but I've fixed it manually. |
* Missing coordinates, * Invalid and nonexistent regridding method, * Invalid coordinate systems.
Additional regridding tests
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.
Looks good to me.
New operators and test to regrid basic rectilinear Geos data using a Linear regridder, with capacity to be expanded to handle other grids/new regrid methods.
Fixes #154