Closed sebffischer closed 1 year ago
Martin already reviewed this here: https://github.com/mlr-org/mlr3pipelines/pull/702
Merging #80 (742b402) into main (3fe73d1) will decrease coverage by
0.36%
. The diff coverage is72.22%
.
@@ Coverage Diff @@
## main #80 +/- ##
==========================================
- Coverage 80.48% 80.11% -0.37%
==========================================
Files 71 71
Lines 999 1016 +17
==========================================
+ Hits 804 814 +10
- Misses 195 202 +7
Impacted Files | Coverage Δ | |
---|---|---|
R/dictionary_sugar.R | 68.11% <72.22%> (+3.41%) |
:arrow_up: |
R/get_seed.R | 60.00% <0.00%> (-40.00%) |
:arrow_down: |
src/keep_in_bounds.c | 94.73% <0.00%> (-5.27%) |
:arrow_down: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
These sugar functions allow do append
_<n>
to the ids of the dictionary to coveniently increment ids. When retrieving the object"pca_1"
this retrieves the object"pca"
and sets its id to"pca_1"
Instead of having to write
it is then possible do write
Note that one wants to use this in a dictionary, one has to ensure that no key satisfies the regex
_\d+$
because then this mechanism is flawed.