Skip to content

Commit

Permalink
fix defaults (#287)
Browse files Browse the repository at this point in the history
  • Loading branch information
sebffischer authored Aug 3, 2023
1 parent a4517ac commit 949cef2
Show file tree
Hide file tree
Showing 8 changed files with 16 additions and 26 deletions.
2 changes: 1 addition & 1 deletion R/learner_C50_classif_C50.R
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ LearnerClassifC50 = R6Class("LearnerClassifC50",
seed = p_int(lower = -Inf, upper = Inf, tags = "train"),
earlyStopping = p_lgl(default = TRUE, tags = "train"),
label = p_uty(default = "outcome", tags = "train"),
na.action = p_uty(default = stats::na.pass, tags = "predict")
na.action = p_uty(default = "stats::na.pass", tags = "predict")
)
ps$add_dep("bands", "rules", CondEqual$new(TRUE))

Expand Down
2 changes: 1 addition & 1 deletion R/learner_lme4_classif_glmer.R
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ LearnerClassifGlmer = R6Class("LearnerClassifGlmer",
re.form = p_uty(default = NULL, tags = "predict"),
random.only = p_lgl(default = FALSE, tags = "predict"),
allow.new.levels = p_lgl(default = FALSE, tags = "predict"),
na.action = p_uty(default = stats::na.pass, tags = "predict")
na.action = p_uty(default = "stats::na.pass", tags = "predict")
)

param_set$values = list(family = stats::binomial(link = "logit"))
Expand Down
7 changes: 2 additions & 5 deletions R/learner_lme4_regr_lmer.R
Original file line number Diff line number Diff line change
Expand Up @@ -118,10 +118,7 @@ LearnerRegrLmer = R6Class("LearnerRegrLmer",
),
ParamUty$new(
id = "check.conv.singular",
default = 'lme4::.makeCC(
action = "message",
tol = formals(lme4::isSingular)$tol
)',
default = 'lme4::.makeCC(action = "message", tol = formals(lme4::isSingular)$tol)',
tags = "train"
),
ParamUty$new(
Expand All @@ -141,7 +138,7 @@ LearnerRegrLmer = R6Class("LearnerRegrLmer",
ParamUty$new(id = "re.form", default = NULL, tags = "predict"),
ParamLgl$new(id = "random.only", default = FALSE, tags = "predict"),
ParamLgl$new(id = "allow.new.levels", default = FALSE, tags = "predict"),
ParamUty$new(id = "na.action", default = stats::na.pass, tags = "predict")
ParamUty$new(id = "na.action", default = "stats::na.pass", tags = "predict")
))

super$initialize(
Expand Down
2 changes: 1 addition & 1 deletion R/learner_partykit_surv_cforest.R
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ delayedAssign(
trace = p_lgl(default = FALSE, tags = "train"),
offset = p_uty(tags = "train"),
cluster = p_uty(tags = "train"),
na.action = p_uty(default = stats::na.pass, tags = "train"),
na.action = p_uty(default = "stats::na.pass", tags = "train"),
scores = p_uty(tags = "train"),

teststat = p_fct(default = "quadratic",
Expand Down
2 changes: 1 addition & 1 deletion man/mlr_learners_classif.C50.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion man/mlr_learners_classif.glmer.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

23 changes: 8 additions & 15 deletions man/mlr_learners_regr.lmer.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion man/mlr_learners_surv.cforest.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 949cef2

Please sign in to comment.