Merge pull request #116 from mlr-org/fix-regr.ksvm

add nu_svr to C parameter
mlr-org · Oct 5, 2021 · fb4976d · fb4976d
2 parents dfe3244 + b9ee305
commit fb4976d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: mlr3extralearners
 Title: Extra Learners For mlr3
-Version: 0.5.8
+Version: 0.5.9
 Authors@R:
     c(person(given = "Raphael",
              family = "Sonabend",

diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,7 @@
+# mlr3extralearners 0.5.9
+
+* Fixed bug that didn't allow C parameter to be set for nu-regression
+
 # mlr3extralearners 0.5.8
 
 * Add `regr.rvm` and `classif.lssvm`

diff --git a/R/learner_kernlab_regr_ksvm.R b/R/learner_kernlab_regr_ksvm.R
@@ -39,7 +39,7 @@ LearnerRegrKSVM = R6Class("LearnerRegrKSVM",
         offset = p_dbl(default = NO_DEF, tags = "train")
       )
 
-      ps$add_dep("C", "type", CondAnyOf$new(c("eps-svr", "eps-bsvr")))
+      ps$add_dep("C", "type", CondAnyOf$new(c("eps-svr", "eps-bsvr", "nu-svr")))
       ps$add_dep("nu", "type", CondAnyOf$new(c("nu-svr")))
       ps$add_dep(
         "epsilon", "type",