mlr3
mlr3 copied to clipboard
BREAKING CHANGE: Introduce default_params
- Improves transparency: Learners should set those values in the initialize function (and not e.g. in $.train)
- Could be used in description for automatic display of defaults
- Could be used to properly reset a learner to it's initial state.
@mllg What do you think?
I like the idea. Would this break mlr3learners and other packages? If yes, can we prepare an intermediate release so that we can migrate gracefully, i.e. without synchronizing CRAN package uploads?
I just realized that we don't even have to break anything, we could simply set
private$.default_params = self$param_set$values
So I think now it should not break anything, and we don't have to change the learners either
should we also automatically reset the parameters to the default parameters when calling $reset()
?