Closed juliohm closed 4 years ago
Merging #134 into master will decrease coverage by
0.16%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #134 +/- ##
==========================================
- Coverage 93.75% 93.58% -0.17%
==========================================
Files 11 10 -1
Lines 560 546 -14
==========================================
- Hits 525 511 -14
Misses 35 35
Impacted Files | Coverage Δ | |
---|---|---|
src/LossFunctions.jl | 100.00% <ø> (ø) |
|
src/supervised.jl | 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 a0a2fa0...a6746ab. Read the comment docs.
This PR removes a single method with
MarginLoss
andSparseArrays
that is hanging in the codebase as a workaround for categorical representations. The plan is to think more carefully about an API withCategoricalArrays
moving forward. This new API should be more explicit about encoding/decoding schemes, possibly avoiding this memory-hungry full one-hot encoding.