Ouranosinc / xclim / 14886240106

92%

Build Type

push

github

Committed by web-flow

Commit Message

Split sdba (#2099)

<!--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:
- [ ] This PR addresses an already opened issue (for bug fixes /
features)
    - This PR fixes #xyz
- [x] Tests for the changes have been added (for bug fixes / features)
- [ ] (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?

* Import `xsdba` instead of having it as a submodule within `xclim`.
This means deleting a lot of code

### Does this PR introduce a breaking change?

Units handling is different in `xsdba`. But perhaps we can patch
`xclim.sdba` to reproduce the previous behaviour?

### Other information:

We must remove `sdba-warning.yml` , yes?

Run Details

8 of 8 new or added lines in 2 files covered. (100.0%)

7 existing lines in 3 files now uncovered.

7554 of 8187 relevant lines covered (92.27%)

8.08 hits per line