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

Mobt 496 enforce forecast consistency #6

Merged
merged 5 commits into from
May 26, 2023

Conversation

brhooper
Copy link
Contributor

Acceptance test data for metoppv/improver#1900

Percentile test data copied from #5
Probability test data copied from old enforce_consistent_probabilities acceptance tests

Copy link
Contributor

@gavinevans gavinevans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @brhooper 👍

I've plotted the difference between the known good outputs and the input forecast below for the two tests, which seem reasonable.

For percentiles (for sites that have a difference), the known good output is larger than the input for some sites for wind gust, which seems reasonable.
Enforce_consistency_percentile_diff

For probabilities, the hybrid total cloud probabilities have been reduced within the known good output, which I think makes sense as the total cloud is acting as a cap.
Enforce_consistency_prob_diff

@gavinevans
Copy link
Contributor

Just as a query, should we consider reducing the number of the percentiles in the percentile forecast tests? Just wondering, if that might help as they're relatively large in terms of file size right now.

@brhooper
Copy link
Contributor Author

I've reduced the percentile files down to 3 percentiles (rather than the original 13). I've plotted the new difference between the forecast and kgo below. As expected, the kgo is greater than the forecast, where changes have been made.

image

@brhooper
Copy link
Contributor Author

An acceptance test has been added which uses realization data. The difference between the kgo and input forecast is shown below. As expected, the kgo is always smaller than the forecast, as it has been enforced to be 90% of the reference forecast (or less).

image

@gavinevans gavinevans merged commit d11c273 into master May 26, 2023
@gavinevans gavinevans deleted the mobt_496_enforce_forecast_consistency branch May 26, 2023 12:40
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.

2 participants