Skip to content

Commit 42d2e3f

Browse files
authored
ci: update no suggests (#195)
* ci: update no suggests * ...
1 parent 8d68945 commit 42d2e3f

15 files changed

+58
-9
lines changed

.github/workflows/no-suggest-cmd-check.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# r cmd check workflow without suggests of the mlr3 ecosystem v0.3.1
22
# https://github.com/mlr-org/actions
3+
# modified to use supercharge/[email protected]
34
on:
45
workflow_dispatch:
56
inputs:
@@ -41,6 +42,10 @@ jobs:
4142
with:
4243
r-version: ${{ matrix.config.r }}
4344

45+
- uses: supercharge/[email protected]
46+
with:
47+
redis-version: 7
48+
4449
- uses: r-lib/actions/setup-r-dependencies@v2
4550
with:
4651
extra-packages: |

R/sugar.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ acqo = function(optimizer, terminator, acq_function = NULL, callbacks = NULL, ..
137137
dots = list(...)
138138

139139
if (is.character(optimizer)) {
140-
return(dictionary_sugar_get(mlr_acqoptimizers, optimizer, ...))
140+
return(dictionary_sugar_get(mlr_acqoptimizers, optimizer, acq_function = acq_function, ...))
141141
}
142142

143143
acqopt = AcqOptimizer$new(optimizer = optimizer, terminator = terminator, acq_function = acq_function, callbacks = callbacks)

tests/testthat/helper.R

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,12 @@ MAKE_DESIGN = function(instance, n = 4L) {
108108
generate_design_random(instance$search_space, n)$data
109109
}
110110

111+
skip_if_missing_regr_km = function() {
112+
skip_if_not_installed("mlr3learners")
113+
skip_if_not_installed("DiceKriging")
114+
skip_if_not_installed("rgenoud")
115+
}
116+
111117
if (requireNamespace("mlr3learners") && requireNamespace("DiceKriging") && requireNamespace("rgenoud")) {
112118
library(mlr3learners)
113119
REGR_KM_NOISY = lrn("regr.km", covtype = "matern3_2", optim.method = "gen", control = list(trace = FALSE), nugget.estim = TRUE, jitter = 1e-12)

tests/testthat/test_AcqFunctionMulti.R

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,8 @@ test_that("AcqFunctionMulti works for single-objective instances", {
2626
})
2727

2828
test_that("AcqFunctionMulti works for multi-objective instances", {
29+
skip_if_not_installed("emoa")
30+
skip_if_not_installed("fastGHQuad")
2931
inst = MAKE_INST(OBJ_1D_2, PS_1D, trm("evals", n_evals = 5L))
3032
surrogate = SurrogateLearnerCollection$new(list(REGR_FEATURELESS, REGR_FEATURELESS$clone(deep = TRUE)), archive = inst$archive)
3133
acqfs = list(AcqFunctionEHVI$new(), AcqFunctionEHVIGH$new())

tests/testthat/test_AcqOptimizerDirect.R

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
test_that("AcqOptimizerDirect works", {
2+
skip_if_missing_regr_km()
23
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 5L))
34
design = generate_design_grid(instance$search_space, resolution = 4L)$data
45
instance$eval_batch(design)
@@ -18,6 +19,7 @@ test_that("AcqOptimizerDirect works", {
1819
})
1920

2021
test_that("AcqOptimizerDirect works with 2D", {
22+
skip_if_missing_regr_km()
2123
instance = oi(OBJ_2D, terminator = trm("evals", n_evals = 5L))
2224
design = generate_design_grid(instance$search_space, resolution = 4L)$data
2325
instance$eval_batch(design)
@@ -37,6 +39,7 @@ test_that("AcqOptimizerDirect works with 2D", {
3739
})
3840

3941
test_that("AcqOptimizerDirect works with instance", {
42+
skip_if_missing_regr_km()
4043
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 10L))
4144
design = generate_design_grid(instance$search_space, resolution = 4L)$data
4245
instance$eval_batch(design)
@@ -51,6 +54,7 @@ test_that("AcqOptimizerDirect works with instance", {
5154
})
5255

5356
test_that("AcqOptimizerDirect works with random restart", {
57+
skip_if_missing_regr_km()
5458
instance = oi(OBJ_2D, terminator = trm("evals", n_evals = 5L))
5559
design = generate_design_grid(instance$search_space, resolution = 4L)$data
5660
instance$eval_batch(design)

tests/testthat/test_AcqOptimizerLbfgsb.R

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
test_that("AcqOptimizerLbfgsb works", {
2+
skip_if_missing_regr_km()
23
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 5L))
34
design = generate_design_grid(instance$search_space, resolution = 4L)$data
45
instance$eval_batch(design)
@@ -18,6 +19,7 @@ test_that("AcqOptimizerLbfgsb works", {
1819
})
1920

2021
test_that("AcqOptimizerLbfgsb works with 2D", {
22+
skip_if_missing_regr_km()
2123
instance = oi(OBJ_2D, terminator = trm("evals", n_evals = 5L))
2224
design = generate_design_grid(instance$search_space, resolution = 4L)$data
2325
instance$eval_batch(design)
@@ -37,6 +39,7 @@ test_that("AcqOptimizerLbfgsb works with 2D", {
3739
})
3840

3941
test_that("AcqOptimizerLbfgsb works with instance", {
42+
skip_if_missing_regr_km()
4043
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 10L))
4144
design = generate_design_grid(instance$search_space, resolution = 4L)$data
4245
instance$eval_batch(design)
@@ -51,6 +54,7 @@ test_that("AcqOptimizerLbfgsb works with instance", {
5154
})
5255

5356
test_that("AcqOptimizerLbfgsb works with random restart", {
57+
skip_if_missing_regr_km()
5458
instance = oi(OBJ_2D, terminator = trm("evals", n_evals = 5L))
5559
design = generate_design_grid(instance$search_space, resolution = 4L)$data
5660
instance$eval_batch(design)

tests/testthat/test_AcqOptimizerLocalSearch.R

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
test_that("AcqOptimizerLocalSearch works", {
2+
skip_if_missing_regr_km()
23
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 5L))
34
design = generate_design_grid(instance$search_space, resolution = 4L)$data
45
instance$eval_batch(design)
@@ -15,6 +16,7 @@ test_that("AcqOptimizerLocalSearch works", {
1516
})
1617

1718
test_that("AcqOptimizerLocalSearch works with 2D", {
19+
skip_if_missing_regr_km()
1820
instance = oi(OBJ_2D, terminator = trm("evals", n_evals = 5L))
1921
design = generate_design_grid(instance$search_space, resolution = 4L)$data
2022
instance$eval_batch(design)
@@ -31,6 +33,7 @@ test_that("AcqOptimizerLocalSearch works with 2D", {
3133
})
3234

3335
test_that("AcqOptimizerLocalSearch works with instance", {
36+
skip_if_missing_regr_km()
3437
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 10L))
3538
design = generate_design_grid(instance$search_space, resolution = 4L)$data
3639
instance$eval_batch(design)

tests/testthat/test_AcqOptimizerRandomSearch.R

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
test_that("AcqOptimizerRandomSearch works", {
2+
skip_if_missing_regr_km()
23
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 5L))
34
design = generate_design_grid(instance$search_space, resolution = 4L)$data
45
instance$eval_batch(design)
@@ -14,6 +15,7 @@ test_that("AcqOptimizerRandomSearch works", {
1415
})
1516

1617
test_that("AcqOptimizerRandomSearch works with 2D", {
18+
skip_if_missing_regr_km()
1719
instance = oi(OBJ_2D, terminator = trm("evals", n_evals = 5L))
1820
design = generate_design_grid(instance$search_space, resolution = 4L)$data
1921
instance$eval_batch(design)
@@ -29,6 +31,7 @@ test_that("AcqOptimizerRandomSearch works with 2D", {
2931
})
3032

3133
test_that("AcqOptimizerRandomSearch works with instance", {
34+
skip_if_missing_regr_km()
3235
instance = oi(OBJ_1D, terminator = trm("evals", n_evals = 10L))
3336
design = generate_design_grid(instance$search_space, resolution = 4L)$data
3437
instance$eval_batch(design)

tests/testthat/test_InputTrafoUnitcube.R

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ test_that("InputTrafoUnitcube works", {
1919
})
2020

2121
test_that("InputTrafoUnitcube works with SurrogateLearner", {
22+
skip_if_missing_regr_km()
2223
instance = MAKE_INST_1D()
2324
design = generate_design_random(instance$search_space, n = 4L)$data
2425
instance$eval_batch(design)
@@ -34,6 +35,7 @@ test_that("InputTrafoUnitcube works with SurrogateLearner", {
3435
})
3536

3637
test_that("InputTrafoUnitcube works with SurrogateLearnerCollection", {
38+
skip_if_missing_regr_km()
3739
instance = MAKE_INST(OBJ_1D_2, search_space = PS_1D, terminator = trm("evals", n_evals = 5L))
3840
design = generate_design_random(instance$search_space, n = 4L)$data
3941
instance$eval_batch(design)
@@ -49,6 +51,7 @@ test_that("InputTrafoUnitcube works with SurrogateLearnerCollection", {
4951
})
5052

5153
test_that("InputTrafoUnitcube works with OptimizerMbo and bayesopt_ego", {
54+
skip_if_missing_regr_km()
5255
instance = MAKE_INST_1D()
5356
surrogate = SurrogateLearner$new(REGR_KM_DETERM)
5457
it = InputTrafoUnitcube$new()
@@ -63,6 +66,7 @@ test_that("InputTrafoUnitcube works with OptimizerMbo and bayesopt_ego", {
6366
})
6467

6568
test_that("InputTrafoUnitcube works with OptimizerMbo and bayesopt_parego", {
69+
skip_if_missing_regr_km()
6670
instance = MAKE_INST(OBJ_1D_2, search_space = PS_1D, terminator = trm("evals", n_evals = 5L))
6771
surrogate = SurrogateLearner$new(REGR_KM_DETERM)
6872
it = InputTrafoUnitcube$new()
@@ -77,6 +81,7 @@ test_that("InputTrafoUnitcube works with OptimizerMbo and bayesopt_parego", {
7781
})
7882

7983
test_that("InputTrafoUnitcube works with OptimizerMbo and bayesopt_smsego", {
84+
skip_if_missing_regr_km()
8085
instance = MAKE_INST(OBJ_1D_2, search_space = PS_1D, terminator = trm("evals", n_evals = 5L))
8186
surrogate = SurrogateLearnerCollection$new(list(REGR_KM_DETERM, REGR_KM_DETERM$clone(deep = TRUE)))
8287
it = InputTrafoUnitcube$new()

tests/testthat/test_OutputTrafoLog.R

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ test_that("OutputTrafoLogworks", {
2727
})
2828

2929
test_that("OutputTrafoLog works with SurrogateLearner", {
30+
skip_if_missing_regr_km()
3031
instance = MAKE_INST_1D()
3132
design = generate_design_random(instance$search_space, n = 4L)$data
3233
instance$eval_batch(design)
@@ -48,6 +49,7 @@ test_that("OutputTrafoLog works with SurrogateLearner", {
4849
})
4950

5051
test_that("OutputTrafoLog works with SurrogateLearnerCollection", {
52+
skip_if_missing_regr_km()
5153
instance = MAKE_INST(OBJ_1D_2, search_space = PS_1D, terminator = trm("evals", n_evals = 5L))
5254
design = generate_design_random(instance$search_space, n = 4L)$data
5355
instance$eval_batch(design)
@@ -69,6 +71,7 @@ test_that("OutputTrafoLog works with SurrogateLearnerCollection", {
6971
})
7072

7173
test_that("OutputTrafoLog works with OptimizerMbo and bayesopt_ego", {
74+
skip_if_missing_regr_km()
7275
instance = MAKE_INST_1D()
7376
surrogate = SurrogateLearner$new(REGR_KM_DETERM)
7477
ot = OutputTrafoLog$new()
@@ -83,6 +86,7 @@ test_that("OutputTrafoLog works with OptimizerMbo and bayesopt_ego", {
8386
})
8487

8588
test_that("OutputTrafoLog works with OptimizerMbo and bayesopt_ego for maximization and EI on log scale", {
89+
skip_if_missing_regr_km()
8690
instance = MAKE_INST(OBJ_1D_MAXIMIZE, search_space = PS_1D, terminator = trm("evals", n_evals = 5L))
8791
surrogate = SurrogateLearner$new(REGR_KM_DETERM)
8892
ot = OutputTrafoLog$new(invert_posterior = FALSE)
@@ -97,6 +101,7 @@ test_that("OutputTrafoLog works with OptimizerMbo and bayesopt_ego for maximizat
97101
})
98102

99103
test_that("OutputTrafoLog works with OptimizerMbo and bayesopt_parego", {
104+
skip_if_missing_regr_km()
100105
instance = MAKE_INST(OBJ_1D_2, search_space = PS_1D, terminator = trm("evals", n_evals = 5L))
101106
surrogate = SurrogateLearner$new(REGR_KM_DETERM)
102107
ot = OutputTrafoLog$new()
@@ -112,6 +117,7 @@ test_that("OutputTrafoLog works with OptimizerMbo and bayesopt_parego", {
112117
})
113118

114119
test_that("OutputTrafoLog works with OptimizerMbo and bayesopt_smsego", {
120+
skip_if_missing_regr_km()
115121
instance = MAKE_INST(OBJ_1D_2, search_space = PS_1D, terminator = trm("evals", n_evals = 5L))
116122
surrogate = SurrogateLearnerCollection$new(list(REGR_KM_DETERM, REGR_KM_DETERM$clone(deep = TRUE)))
117123
ot = OutputTrafoLog$new()

0 commit comments

Comments
 (0)