mb706

additionally, maybe we want to give the user an option to set a total timeout for learners instead of setting train_timeout and predict_timeout. Internally prediction timeout should then be the...

no, this is just from changing the API of R6-objects. In S3 this is less of an issue; if the data layout does not change then "old" objects can be...

it would be nicer to work with R's own serialization because that also works when the task is "hidden" somewhere in a list or environment of a function (e.g. objective...

FYI this is known internally, but idk if / where we have the issue about this. It's just broken, don't load old and new mlr (or paradox & ParamHelpers) in...

I am reordering the whole chapter, so no problem with merge conflicts

Still a draft, sorry, I will try to get this in shape tomorrow

Can be reviewed now; I don't know why it is not rendering, to me it doesn't look like I touched anything that should give the error that occurs. It renders...

good to hear

I also stumbled over this and had wondered if it is a bug. I'm wondering why the initial line ("`... -n_bad...`") was changed, since that one does what it says...

Addendum: [this](https://github.com/mlr-org/mlrMBO/blob/7ca9ad456e18024f8d1d1cacfe8f1a3ad3a67360/R/infillOptFocus.R#L58-L59) is a bug anyway since `to.del` is a number and always ranges from 1 to `length(par$values) - 1`, no matter what `val` was.