search

mlr-org / mlr3tuning

Hyperparameter optimization package of the mlr3 ecosystem
https://mlr3tuning.mlr-org.com/
GNU Lesser General Public License v3.0
53 stars 5 forks source link

remove TuneToken for Learners (with tests) #282

Closed mb706 closed 3 years ago

mb706 commented 3 years ago

with tests

jakob-r commented 3 years ago

I don't like that approach so much because what if tune tokens are in the learner but the search_space is actually created with different params. We would have to catch that in a weird place. I thin a clean API between Paradox and mlr3tuning would hide the existence of TuneTokens to mlr3tuning.

Lets keep the discussion in the main thread

be-marc commented 3 years ago

This is fixed now.