Ouranosinc / xsdba / 21526505102

push

github

Address warnings, refactor grouped_time_indexes (#250)

### What kind of change does this PR introduce?

* Addresses warnings stemming from the use of now deprecated
`xarray.cftime_range` (now `xarray.date_range(..., use_cftime=True)`).
* Adds an import check for `numpy.trapezoid`
* Ignores a division by zero `RuntimeWarning` from `numpy` raised by
tests (as is expected).
* Refactors `grouped_time_indexes` to use `GroupBy.map` instead of
`GroupBy.apply`, as well as moderately adjusts internal logic of the
function.

### Does this PR introduce a breaking change?

It should not.

### Other information:

The `grouped_time_indexes` function was refactored with the aid of AI.

16 of 28 new or added lines in 2 files covered. (57.14%)

2805 of 3577 relevant lines covered (78.42%)

5.37 hits per line