1
90%
main: 92%

Ran 16 Feb 2024 05:38PM UTC

Files 70

Run time 2s

Badge

Embed ▾

Committed 16 Feb 2024 05:25PM UTC coverage: 89.853% (-0.02%) from 89.869%

Job # 7934069860.1

Build Type

push

github

Committed by

web-flow

Commit Message

Remove lmoments3 (#1644)

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

* Remove `lmoments3` from xclim's dependencies.
* Accept `rv_continuous` instances for the `dist` arguments of the
statistical indices

### Does this PR introduce a breaking change?

Yes. Passing a mere string and `method='PWM'` is now broken. It will
raise a ValueError with a message asking to pass an instance of
`lmoments3` instead.

This also makes some functions awkward to use. Before, we relied on the
`scipy_dist` attribute of `params` to retrieve the distribution when
computing statistics. As one can now pass an object that has nothing to
do with scipy, those functions (`parametric_quantile` for example) must
now also accept the dist as an argument.

### Other information:

See Ouranosinc/lmoments3#12.

I kinda cheated and made it so the `dist` argument would show up as a
"String" parameter to the indicator. This only changes the metadata,
distributions objects can still be passed to indicators.

Run Details

8598 of 9569 relevant lines covered (89.85%)

1.79 hits per line

Ouranosinc / xclim / 7934069860 / 1
90%
main: 92%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job run-{{ matrix.tox-env }}-opt-slow - 7934069860.1

Ouranosinc / xclim / 7934069860 / 1 90% main: 92%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job run-{{ matrix.tox-env }}-opt-slow - 7934069860.1

Ouranosinc / xclim / 7934069860 / 1
90%
main: 92%

README BADGES
x