codecov[bot]
commented
2 years ago Codecov Report
Merging #706 (0dd622f) into main (47175c8) will not change coverage. The diff coverage is
100.00%.
@@ Coverage Diff @@
## main #706 +/- ##
=======================================
Coverage 96.89% 96.89%
=======================================
Files 63 63
Lines 9199 9199
=======================================
Hits 8913 8913
Misses 286 286 | Impacted Files | Coverage Δ | |
|---|---|---|
| skll/learner/voting.py | 98.45% <ø> (ø) |
|
| tests/test_preprocessing.py | 100.00% <ø> (ø) |
|
| tests/test_voting_learners_expts_1.py | 98.38% <ø> (ø) |
|
| tests/test_voting_learners_expts_2.py | 98.82% <ø> (ø) |
|
| tests/test_voting_learners_expts_3.py | 98.82% <ø> (ø) |
|
| tests/test_voting_learners_expts_4.py | 98.76% <ø> (ø) |
|
| tests/test_voting_learners_expts_5.py | 98.48% <ø> (ø) |
|
| skll/config/__init__.py | 95.13% <100.00%> (ø) |
|
| skll/experiments/__init__.py | 95.14% <100.00%> (ø) |
|
| skll/learner/__init__.py | 97.13% <100.00%> (ø) |
|
| ... and 6 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 47175c8...0dd622f. Read the comment docs.
This PR closes #569.
We are only doing part of the issue here (the renaming) but not moving the
pos_labelfield into theInputsection from theTuningsection where it currently lives. This is mainly to minimize the amount of changes that someone would need to go through when converting their old code/config files (renaming can be automated).