4
92%
main: 92%

Ran 31 Oct 2024 02:45PM UTC

Files 73

Run time 4s

Badge

Embed ▾

Committed 31 Oct 2024 02:32PM UTC coverage: 83.542%. Remained the same

Job # 11613924792.4

Build Type

push

github

Committed by

web-flow

Commit Message

Fix injection of added parameters (#1981)

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

The issue was that a YAML definition of an indicator was not able to
inject (set) the `indexer` parameter on indicators that inherited from
`IndexingIndicator` (i.e. indicators that rely on a class-level
implementation of the indexing, instead of compute-level one).

This reorganizes how these class-added arguments are handled. It also
renames them "added" arguments instead of "injected" to clear the
confusion. "injection" now only means an indicator argument that is set
when creating a derived indicator (that what most YAML definitions do).
"added" is an argument that is not in the compute function, but added by
the class.

The example YAML is modified and a test is added using this
modification.

### Does this PR introduce a breaking change?
No.

The feature was not really documented and now it works instead of not.

Run Details

8761 of 10487 relevant lines covered (83.54%)

0.84 hits per line

Ouranosinc / xclim / 11613924792 / 4
92%
main: 92%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job run-3.11-ubuntu-latest-py311-coverage-sbck-offline-prefetch - 11613924792.4

Ouranosinc / xclim / 11613924792 / 4 92% main: 92%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job run-3.11-ubuntu-latest-py311-coverage-sbck-offline-prefetch - 11613924792.4

Ouranosinc / xclim / 11613924792 / 4
92%
main: 92%

README BADGES
x