Jeremy Coyle

I think this works for learners where the goal is predicting an outcome. However, it's not clear how this would work for "learners" where the goal is something else, for...

By making it `nrow` instead of `length`, you're enforcing that predict returns something that's matrix shaped. Is that what you want? This seems like it might break user expectations, as...

okay, it sounds like the easiest thing is to change it to `sl3::safe_dim(x)[1]` for now, and then work on enforcing prediction types as a later effort

There seems to be a recent bug in `sl3` that prevents time series super learner from working correctly. Thanks for reporting this. We'll get it fixed ASAP

This should now be fixed on devel. You can install the devel version by doing install_github("tlverse/sl3@devel"). It will be merged up to master shortly.

Great question (thanks for all your high-quality feedback lately). As I understand it, you're essentially using `caret` to do a discrete Super Learner over a grid of hyperparameter values for...

I couldn't make this work, even with the extra step. I made an issue on the dbarts repo: https://github.com/vdorie/dbarts/issues/19 I'll complete this once that's resolved

Good to know. Did you get the warning mentioned in that future issue when you tried multicore in RStudio?

The number of threads was based on the machine used to run the vignette. That said, in my experience, i've found that even hyperthreading (using 2 threads/core) either has no...

Unfortunately, we haven't prioritized this task, although I agree we should follow up on it. If you're not already doing so, setting `keepExtra=FALSE` for `Lrnr_sl` will save a lot of...