mlr-org / mlr3learners

Recommended learners for mlr3
https://mlr3learners.mlr-org.com
GNU Lesser General Public License v3.0
89 stars 14 forks source link

fix: cleanup after unloading + doc fixes #258

Closed sebffischer closed 1 year ago

sebffischer commented 1 year ago

This addresses the comments mentioned here: https://github.com/mlr-org/mlr3learners/issues/257

It also cleans up the dictionary after unloading mlr3learners, which was observed here: https://github.com/mlr-org/mlr3website/issues/75

codecov-commenter commented 1 year ago

Codecov Report

Merging #258 (a80fd19) into main (cb8162a) will decrease coverage by 0.04%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #258      +/-   ##
==========================================
- Coverage   96.80%   96.75%   -0.05%     
==========================================
  Files          25       25              
  Lines        1376     1358      -18     
==========================================
- Hits         1332     1314      -18     
  Misses         44       44              
Impacted Files Coverage Δ
R/LearnerClassifCVGlmnet.R 100.00% <ø> (ø)
R/LearnerClassifGlmnet.R 100.00% <ø> (ø)
R/LearnerClassifKKNN.R 100.00% <ø> (ø)
R/LearnerClassifLDA.R 100.00% <ø> (ø)
R/LearnerClassifLogReg.R 95.00% <ø> (ø)
R/LearnerClassifMultinom.R 94.00% <ø> (ø)
R/LearnerClassifNaiveBayes.R 100.00% <ø> (ø)
R/LearnerClassifNnet.R 100.00% <ø> (ø)
R/LearnerClassifQDA.R 100.00% <ø> (ø)
R/LearnerClassifRanger.R 91.35% <ø> (ø)
... and 12 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.