Skip to content
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

713 risk matrix (under development) #724

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

rob-taggart
Copy link
Collaborator

@rob-taggart rob-taggart commented Nov 1, 2024

Please work through the following checklists. Delete anything that isn't relevant.

Development for new xarray-based metrics

  • Works with n-dimensional data and includes reduce_dims, preserve_dims, and weights args.
  • Typehints added
  • Docstrings complete and followed Napoleon (google) style
  • Reference to paper/webpage is in docstring
  • Add error handling
  • Imported into the API

Testing of new xarray-based metrics

  • 100% unit test coverage
  • Test that metric is compatible with dask.
  • Test that metrics work with inputs that contain NaNs
  • Test that broadcasting with xarray works
  • Test both reduce and preserve dims arguments work
  • Test that errors are raised as expected
  • Test that it works with both xr.Dataarrays and xr.Datasets

Tutorial notebook

  • Short introduction to why you would use that metric and what it tells you
  • A link to a reference
  • A "things to try next" section at the end
  • Add notebook to Tutorial_Gallery.ipynb
  • Optional - a detailed discussion of how the metric works at the end of the notebook

Documentation

@rob-taggart rob-taggart changed the title 713 risk matrix 713 risk matrix (under development) Nov 1, 2024
@rob-taggart
Copy link
Collaborator Author

To do:

  • examples in docstring
  • formulae in docstring
  • datasets in tests, plus update type hints
  • dask

@rob-taggart
Copy link
Collaborator Author

@nicholasloveday , this PR is ready for review now, including a tutorial. I've uploaded a version of the paper, which explains the score. Happy to talk you through it before you review so you know the salient bits.

warning_verification.pdf

@rob-taggart
Copy link
Collaborator Author

@nicholasloveday , one thing I didn't do is link the tutorial to the API. Will need to address this in the review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant