[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.
Pull Request Checklist:
number
) and pull request (:pull:number
) has been added.What kind of change does this PR introduce?
produce_horizon
.op
inproduce_horizon
.compute_indicators
where thecat: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 akwarg
, since it is now optional.Other information: