-
-
Notifications
You must be signed in to change notification settings - Fork 602
Assertion issue in Chapter 12 (hypertuning) #1150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@maltenform It seems to be working when you remove progressr::with_progress(expr = {
rr_spcv_svm = mlr3::resample(task = task,
learner = at_ksvm,
# outer resampling (performance level)
resampling = perf_level,
store_models = FALSE)
}) |
And so it does work! Have it running now. Thank you! |
Well, leaving out the encapsulation parameter will use the default which is |
@jannes-m I do not fully understand the discussion in the related issue... Do we need to change anything in the book? |
Hey @Nowosad, yes, we need to change something. Actually I wanted to wait until this is solved upstream. But since this might still take a while, I can also implement the solution as suggested in mlr-org/mlr3#1249 |
I'm running the hyperparameter code:
progressr::with_progress(expr = {
rr_spcv_svm = mlr3::resample(task = task,
learner = at_ksvm,
# outer resampling (performance level)
resampling = perf_level,
store_models = FALSE,
encapsulate = "evaluate")
})
and I get
Error in assert_learner(fallback, task_type = self$task_type) :
Assertion on 'fallback' failed: Must inherit from class 'Learner', but has class 'NULL'.
I saw #1110 and tried downgrading my versions (to 0.90 of extralearners and 21.1 of mlr3, as the 0.90 of extra learners said it needed at least 21.1) but no success.
Is this a bug with a known fix?
If it helps, I'm using R 4.4.1 on Mac running 15.2
The text was updated successfully, but these errors were encountered: