Closed ablaom closed 1 year ago
Merging #141 (7109243) into dev (9d98288) will not change coverage. The diff coverage is
n/a
.
:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@ Coverage Diff @@
## dev #141 +/- ##
=======================================
Coverage 95.28% 95.28%
=======================================
Files 21 21
Lines 869 869
=======================================
Hits 828 828
Misses 41 41
Impacted Files | Coverage Δ | |
---|---|---|
src/mlj/classifiers.jl | 100.00% <ø> (ø) |
|
src/mlj/regressors.jl | 12.50% <ø> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
In making #138 I overlooked that fact that solver switching is not based on the literal value of
gamma
but on the value of thepenalty
parameter (taking values:l1
,:l2
, and:en
) which I had completely overlooked. This PR addresses this mistake and makes further changes to address the observation made here.