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
datamodel: add __check_metrics__()
#301
datamodel: add __check_metrics__()
#301
Conversation
Add helpfer function to determine if the select metrics are valid (e.g. if evaluating a deterministic forecast, then should use deterministic forecast metrics, not probabilistic).
__check_metrics__()
__check_metrics__()
Remove the probabilistic forecasts from `__check_metrics__()` since they are not yet fully integrated into the `metrics.calculator` code.
@wholmgren This PR (adding the |
Looks good assuming we add follow up issue for probabilistic metrics. I think it's worth a quick note in the whats new file too. |
Just opened issue #302 for checking probabilistic metrics and added a line to |
@alorenzo175 ok to merge? |
docs/source/api.rst
for API changes.docs/source/whatsnew
for all changes. Includes link to the GitHub Issue with:issue:`num`
or this Pull Request with:pull:`num`
. Includes contributor name and/or GitHub username (link with:ghuser:`user`
).Adds helper function
__check_metrics__()
to determine if the select metrics are valid (e.g. if evaluating a deterministic forecast, then should use deterministic forecast metrics, not probabilistic).