7934069860
90%
main: 92%

Ran 16 Feb 2024 05:31PM UTC

Jobs 2

Files 70

Run time 12s

Badge

Embed ▾

Committed 16 Feb 2024 05:25PM UTC coverage: 90.187% (+0.004%) from 90.183%

Build # 7934069860

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

8630 of 9569 relevant lines covered (90.19%)

6.97 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	run-{{ matrix.tox-env }}-opt-slow - 7934069860.1	16 Feb 2024 05:38PM UTC	0	89.85
2	run-{{ matrix.tox-env }}-{{ matrix.os }} - 7934069860.2	16 Feb 2024 05:38PM UTC	0	90.19

Source Files on build 7934069860

Detailed source file information is not available for this build.

Ouranosinc / xclim / 7934069860
90%
main: 92%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 7934069860

Ouranosinc / xclim / 7934069860 90% main: 92%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 7934069860

Ouranosinc / xclim / 7934069860
90%
main: 92%

README BADGES
x