codecov[bot]
commented
3 years ago Codecov Report
Merging #50 (52c6b0c) into main (6150175) will decrease coverage by
0.33%. The diff coverage is85.71%.:exclamation: Current head 52c6b0c differs from pull request most recent head f5c643d. Consider uploading reports for the commit f5c643d to get more accurate results
@@ Coverage Diff @@
## main #50 +/- ##
==========================================
- Coverage 98.93% 98.60% -0.34%
==========================================
Files 7 9 +2
Lines 283 287 +4
==========================================
+ Hits 280 283 +3
- Misses 3 4 +1 | Impacted Files | Coverage Δ | |
|---|---|---|
| src/AxisSets.jl | 75.00% <ø> (ø) |
|
| src/impute.jl | 100.00% <ø> (ø) |
|
| src/utils.jl | 75.00% <75.00%> (ø) |
|
| src/featuretransforms.jl | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 6150175...f5c643d. Read the comment docs.
branched off https://github.com/invenia/AxisSets.jl/tree/bc/featuretransforms and partially replaces #44
apply!andapply_appendmethods for now since all we need isapplyfor KeyedDataset to beis_transformablekeyargument for similar reasons - now it looks more likeImpute.applymethod so I also replaced the original_patternmethods.