-
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
Add functionality to filter by region #522
Conversation
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 good. I've made some small suggestions and asked a couple questions, but they shouldn't take long.
Once their addressed hit me up for a re-review and I can go ahead and approve it.
cset-workflow/includes/lfric_deterministic_domain_mean_surface_time_series.cylc
Outdated
Show resolved
Hide resolved
cset-workflow/includes/lfric_deterministic_domain_mean_surface_time_series.cylc
Outdated
Show resolved
Hide resolved
cset-workflow/includes/lfric_plot_spatial_surface_model_field.cylc
Outdated
Show resolved
Hide resolved
cset-workflow/includes/lfric_plot_spatial_surface_model_field.cylc
Outdated
Show resolved
Hide resolved
@jfrost-mo think I'm done, let me know if you've got any other comments |
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.
Thanks for addressing the feedback. The only question I've got left is what is tests/test_data/air_temp.nc.aux.xml used for? I can't find any other references to it in the code.
I'll go ahead and merge then. |
fba2732
to
a9d541a
Compare
…_time_series.cylc Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
…_time_series.cylc Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
…cylc Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
…cylc Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
a9d541a
to
54a1b6c
Compare
Description
This branch enables the user to filter the data by geographic region defined in a shapefile (geospatial vector data) format.
Fixes #400
Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.
docs/source/changelog.rst