Merge pull request #917 from mlr-org/depends_only_checks

check with depends only

Author: mb706

.github/workflows/r-cmd-check.yml

@@ -15,7 +15,7 @@ jobs:
   r-cmd-check:
     runs-on: ${{ matrix.config.os }}
 
-    name: ${{ matrix.config.os }} (${{ matrix.config.r }})
+    name: ${{ matrix.config.os }} (${{ matrix.config.r }})${{ matrix.config.depends_only && ' – noSuggests' || '' }}
 
     env:
       GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
@@ -26,6 +26,7 @@ jobs:
         config:
           - {os: ubuntu-latest,   r: 'devel'}
           - {os: ubuntu-latest,   r: 'release'}
+          - {os: ubuntu-latest,   r: 'release', depends_only: true}
 
     steps:
       - uses: actions/checkout@v4
@@ -40,3 +41,6 @@ jobs:
           needs: check
 
       - uses: r-lib/actions/check-r-package@v2
+        env:
+          _R_CHECK_DEPENDS_ONLY_: ${{ matrix.config.depends_only && 'TRUE' || '' }}
+          NOT_CRAN: ${{ matrix.config.depends_only && 'FALSE' || 'TRUE' }}

DESCRIPTION

@@ -109,7 +109,7 @@ Config/testthat/parallel: true
 NeedsCompilation: no
 Roxygen: list(markdown = TRUE, r6 = FALSE)
 RoxygenNote: 7.3.2
-VignetteBuilder: knitr
+VignetteBuilder: knitr, rmarkdown
 Collate:
     'CnfAtom.R'
     'CnfClause.R'

R/preproc.R

@@ -42,7 +42,7 @@
 #' of [`TaskSupervised`][mlr3::TaskSupervised] will not work with these input types for `indata`.
 #'
 #' @export
-#' @examples
+#' @examplesIf requireNamespace("rpart")
 #' library("mlr3")
 #' task = tsk("iris")
 #' pop = po("pca")

man/preproc.Rd

@@ -938,6 +938,7 @@ test_that("GraphLearner hashes", {
 })
 
 test_that("validation, internal_valid_scores", {
+  skip_if_not_installed("rpart")
   expect_error(as_pipeop(lrn("classif.debug", validate = 0.3)), "must either be")
   # None of the Learners can do validation -> NULL
   glrn1 = as_learner(as_graph(lrn("classif.rpart")))$train(tsk("iris"))
@@ -966,6 +967,7 @@ test_that("validation, internal_valid_scores", {
 })
 
 test_that("internal_tuned_values", {
+  skip_if_not_installed("rpart")
   # no internal tuning support -> NULL
   task = tsk("iris")
   glrn1 = as_learner(as_graph(lrn("classif.rpart")))$train(task)
@@ -1074,6 +1076,7 @@ test_that("marshal", {
 })
 
 test_that("marshal has no effect when nothing needed marshaling", {
+  skip_if_not_installed("rpart")
   task = tsk("iris")
   glrn = as_learner(as_graph(lrn("classif.rpart")))
   glrn$train(task)

tests/testthat/test_GraphLearner.R

@@ -140,6 +140,8 @@ test_that("marshal", {
 })
 
 test_that("marshal multiplicity", {
+  skip_if_not_installed("rpart")
+  skip_if_not_installed("bbotk")
   po = po("learner_cv", learner = lrn("classif.debug"))
   po$train(list(Multiplicity(tsk("iris"), tsk("sonar"))))
   s = po$state

tests/testthat/test_pipeop_learnercv.R

@@ -240,7 +240,8 @@ test_that("threshold works with cost measure", {
 
 
 test_that("threshold graph transparency", {
-
+  skip_if_not_installed("rpart")
+  skip_if_not_installed("bbotk")
   lrn_prob = as_learner(
     po("learner_cv", lrn("classif.rpart", predict_type = "prob")) %>>%
       po("tunethreshold")

tests/testthat/test_pipeop_tunethreshold.R

@@ -1,6 +1,7 @@
 context("preproc")
 
 test_that("preproc - basic sanity checks", {
+  skip_if_not_installed("smotefamily")
   task = tsk("iris")
   op = po("scale")
   graph = as_graph(op)