-
Notifications
You must be signed in to change notification settings - Fork 2
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 between references #12
Mobt 496 enforce forecast between references #12
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.
Most of the changes are just name changes, which should be fine. I've just the acceptance tests with the new acceptance test data and that all looks fine. However when I tried to look at the output for the new file (double_bound_percentile_kgo.nc) I couldn't see any data. I'm not sure whether it is something I've done, so could you check what it looks like for you?
Thanks for the review @Katie-Howard. Regarding your question, I seem to be able to see data in that kgo. I've plotted the double_bounds_percentile_kgo.nc and single_bounds_percentile_kgo.nc below, as well as the difference between them (which seems to be restricted to one site hidden in the Alps). |
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've checked through this and it all looks fine 👍
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.
This all looks good to me.
Adds new kgo for acceptance test introduced in metoppv/improver#1907