Ouranosinc / xsdba / 21601847137

79%

push

github

fix adapt-freq: Check for negative OR null values of `dP0` (#249)

<!-- 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 #xyz
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features).
- [x] (If applicable) Tests have been added.
- [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?

* Fix an issue where adapt-freq had a weird mix for -inf / nan, causing
again the insertion of some nans in adapted precips

### Does this PR introduce a breaking change?

kind of, fixes wrong behaviour

### Other information:

Sorry for the name of the branch, I got confused

3 of 3 new or added lines in 1 file covered. (100.0%)

2805 of 3577 relevant lines covered (78.42%)

5.36 hits per line