Ouranosinc / xclim / 35109169469

91%

push

github

Add multimember test (#2411)

<!--Please ensure the PR fulfills the following requirements! -->
<!-- If this is your first PR, make sure to add your details to the
AUTHORS.rst! -->
### Pull Request Checklist:
- [x] This PR addresses an already opened issue (for bug fixes /
features)
    - This PR fixes #2406 
- [x] Tests for the changes have been added (for bug fixes / features)
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features)
- [x] CHANGELOG.rst has been updated (with summary of main changes)
- [x] Link to issue (:issue:`number`) and pull request (:pull:`number`)
has been added

### What kind of change does this PR introduce?

* Add multimember test for robustness_fractions.
* the input is the same a before ( a realization dim with everything).
`stack_ensemble_member` finds the members pattern and the rest of the id
(which I call sub_id) is considered the new realization.
* Previous test only change computation inside of the test, but if this
test is used, it also affects the computation of `delta` and `valid`
which are now first reduce on the member dimension.
* fix #2406 
* Change a pre-commit url for original version instead of mirror

### Does this PR introduce a breaking change?
yes, for ipcc-ar6-c test the previous results were wrong


### Other information:
math showing that multimember is the same as ipcc-ar6-c but with less
assumptions.
<img width="1095" height="615" alt="Capture d’écran, le 2026-08-31 à
10 38 41"
src="https://github.com/user-attachments/assets/de1ce1db-3482-4428-8a57-b5ae45f0c34f"
/>

120 of 129 new or added lines in 1 file covered. (93.02%)

8081 of 8909 relevant lines covered (90.71%)

7.65 hits per line