Skip to content

Commit 44cd667

Browse files
author
Raphael
committed
patches
* Fixed ranger and rpart parameters * Move SVM to suggests * Fixed UnoC documentation
1 parent abbc521 commit 44cd667

File tree

10 files changed

+13
-30
lines changed

10 files changed

+13
-30
lines changed

DESCRIPTION

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,7 @@ Imports:
4242
mlr3pipelines,
4343
paradox (>= 0.1.0),
4444
R6,
45-
survival,
46-
survivalsvm
45+
survival
4746
Suggests:
4847
flexsurv,
4948
gbm,
@@ -59,6 +58,7 @@ Suggests:
5958
rpart,
6059
simsurv,
6160
survAUC,
61+
survivalsvm,
6262
testthat
6363
VignetteBuilder:
6464
knitr

NAMESPACE

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,6 @@ importFrom(stats,model.frame)
6363
importFrom(stats,model.matrix)
6464
importFrom(stats,reformulate)
6565
importFrom(survival,Surv)
66-
importFrom(survivalsvm,makediff1)
67-
importFrom(survivalsvm,makediff2)
68-
importFrom(survivalsvm,makediff3)
6966
importFrom(utils,data)
7067
importFrom(utils,head)
7168
importFrom(utils,tail)

NEWS.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,10 @@
33
* Added `method` argument to integrated scores and added weighting by bin-width
44
* Added notes to IGS documentation regarding default methods and comparison to other packages
55
* Added `method` to `MeasureSurvIntegrated` constructor and fields
6-
* Fixed mistake in `TaskSurv` documentation
6+
* Fixed mistake in documentation of: `TaskSurv`, `MeasureSurvUnoC`
77
* Fixed errors in r-patched-solaris
8+
* Added missing `LearnerSurvRpart` parameter `parms` and `cost`
9+
810

911
# mlr3proba 0.1.0
1012

R/LearnerSurvRanger.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ LearnerSurvRanger = R6Class("LearnerSurvRanger", inherit = LearnerSurv,
3131
ParamLgl$new(id = "replace", default = TRUE, tags = "train"),
3232
ParamDbl$new(id = "sample.fraction", lower = 0L, upper = 1L, tags = "train"), # for replace == FALSE, def = 0.632
3333
# ParamDbl$new(id = "class.weights", defaul = NULL, tags = "train"), #
34-
ParamFct$new(id = "splitrule", levels = c("variance", "extratrees", "maxstat"), default = "variance", tags = "train"),
34+
ParamFct$new(id = "splitrule", levels = c("logrank","extratrees","C","maxstat"), default = "logrank", tags = "train"),
3535
ParamInt$new(id = "num.random.splits", lower = 1L, default = 1L, tags = "train"), # requires = quote(splitrule == "extratrees")
3636
ParamDbl$new(id = "split.select.weights", lower = 0, upper = 1, tags = "train"),
3737
ParamUty$new(id = "always.split.variables", tags = "train"),

R/LearnerSurvRpart.R

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,14 @@ LearnerSurvRpart = R6Class("LearnerSurvRpart", inherit = LearnerSurv,
1818
public = list(
1919
initialize = function() {
2020
ps = ParamSet$new(list(
21+
ParamDbl$new("parms", default = 1, tags = "train"),
2122
ParamInt$new("minsplit", default = 20L, lower = 1L, tags = "train"),
2223
ParamDbl$new("cp", default = 0.01, lower = 0, upper = 1, tags = "train"),
2324
ParamInt$new("maxcompete", default = 4L, lower = 0L, tags = "train"),
2425
ParamInt$new("maxsurrogate", default = 5L, lower = 0L, tags = "train"),
2526
ParamInt$new("maxdepth", default = 30L, lower = 1L, upper = 30L, tags = "train"),
26-
ParamInt$new("xval", default = 10L, lower = 0L, tags = "train")
27+
ParamInt$new("xval", default = 10L, lower = 0L, tags = "train"),
28+
ParamUty$new("cost", tags = "train")
2729
))
2830
ps$values = list(xval = 0L)
2931

R/LearnerSurvSVM.R

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
#' @importFrom survivalsvm makediff1 makediff2 makediff3
21
#' @template surv_learner
32
#' @templateVar title Regression, Ranking and Hybrid Support Vector Machines
43
#' @templateVar fullname LearnerSurvSVM
@@ -25,12 +24,6 @@
2524
#' Artificial Intelligence in Medicine, 53, 107-118.
2625
#'
2726
#' @export
28-
#' @examples
29-
#' library(mlr3)
30-
#' task = tgen("simsurv")$generate(20)
31-
#' learner = lrn("surv.svm")
32-
#' resampling = rsmp("cv", folds = 2)
33-
#' resample(task, learner, resampling)
3427
LearnerSurvSVM = R6Class("LearnerSurvSVM", inherit = LearnerSurv,
3528
public = list(
3629
initialize = function() {

R/MeasureSurvUnoC.R

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,6 @@
33
#' @templateVar inherit [MeasureSurv]
44
#' @templateVar fullname MeasureSurvUnoC
55
#' @templateVar shortname surv.unoC
6-
#' @templateVar pars times, lp_thresh
7-
#' @templateVar times_par TRUE
8-
#' @templateVar thresh_par TRUE
96
#'
107
#' @description
118
#' Calls [survAUC::UnoC()].

man/LearnerSurvSVM.Rd

Lines changed: 0 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

man/MeasureSurvUnoC.Rd

Lines changed: 1 addition & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/testthat/test_mlr_learners_surv_svm.R

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
context("surv.svm")
22

33
test_that("autotest", {
4+
makediff1 = survivalsvm::makediff1
5+
makediff2 = survivalsvm::makediff2
6+
makediff3 = survivalsvm::makediff3
47
learner = mlr_learners$get("surv.svm")
58
expect_learner(learner)
69
result = run_autotest(learner,exclude = "feat_single")

0 commit comments

Comments
 (0)