Ouranosinc / xclim / 33083570671

91%

push

github

Allow open bounds in select_time (#2374)

<!--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 #2365 
- [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?

* Rename ``xclim.core.calendar.mask_between_doys`` to
``xclim.core.calendar.select_between_doys``
* Allow open ``date_bounds`` and ``doy_bounds`` (i.e., ``None``) in
``xclim.core.calendar.select_time``
* Add ``include_doy_bounds_nans`` and ``bounds_freq`` parameters to
``select_time`` to specify how NaN should be handle in ``doy_bounds``
and the start of the period, respectively.

### Does this PR introduce a breaking change?
The minor introduced breaking change is that array-like ``doy_bounds``
without a ``time`` dimension must now be consecutive according to the
frequency (it was already the case for ``DataArray`` with ``time``).

### Other information:

59 of 60 new or added lines in 1 file covered. (98.33%)

7947 of 8756 relevant lines covered (90.76%)

7.64 hits per line