Ouranosinc / xscen / 11669443993

90%

push

github

More flexible `produce_horizon` (#483)

<!-- 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] This PR does not seem to break the templates.
- [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?

* You can now skip the computation of indicators, if you've already done
that step and just want the subsetting/climatological_op utilities of
`produce_horizon`.
* You can now control the `op` in `produce_horizon`.
* Fixed a bug in `compute_indicators` where the `cat:variable` was
determined too early and only gave the first indicator even when
multiple were computed.

### Does this PR introduce a breaking change?

- `indicators` was moved to a `kwarg`, since it is now optional.


### Other information:

6 of 7 new or added lines in 2 files covered. (85.71%)

3591 of 4411 relevant lines covered (81.41%)

4.79 hits per line