Closed ablaom closed 2 years ago
Okay. Looks like a testing dependency issue. We need:
Merging #127 (fa98744) into master (5684782) will decrease coverage by
6.03%
. The diff coverage is78.94%
.
@@ Coverage Diff @@
## master #127 +/- ##
==========================================
- Coverage 93.87% 87.83% -6.04%
==========================================
Files 8 8
Lines 245 296 +51
==========================================
+ Hits 230 260 +30
- Misses 15 36 +21
Impacted Files | Coverage Δ | |
---|---|---|
src/model_api.jl | 70.00% <50.00%> (+3.33%) |
:arrow_up: |
src/model_traits.jl | 71.69% <62.16%> (-1.48%) |
:arrow_down: |
src/data_utils.jl | 80.64% <79.22%> (-17.57%) |
:arrow_down: |
src/MLJModelInterface.jl | 100.00% <100.00%> (ø) |
|
src/equality.jl | 100.00% <100.00%> (ø) |
|
src/metadata_utils.jl | 100.00% <100.00%> (ø) |
|
src/model_def.jl | 100.00% <100.00%> (ø) |
|
src/parameter_inspection.jl | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 5684782...fa98744. Read the comment docs.
126