Commit 4410a74f authored by Thibault Hallouin's avatar Thibault Hallouin
Browse files

harmonise definitions for above/below threshold events

Low flow events were defined as the complement of high flow events,
meaning that Brier scores were symmetric. But it is more consistent to
include the threshold value in both the definitions of low flow and
high flow events since we either study one or the other, and rarely
both at the same time, especially not using the same threshold values.
Plus, the choice of including the threshold in high flow events rather
than low flow ones was really arbitrary.

Note, only the unit tests for contingency table-based metrics are
impacted because threshold-based metrics are using evants="high".
1 merge request!3release v0.1.0
Pipeline #44110 failed with stage
in 3 minutes and 15 seconds
Showing with 211 additions and 211 deletions
+211 -211
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment