search

a-hanf / mlr3automl

Automated machine learning in mlr3
GNU Lesser General Public License v3.0
25 stars 4 forks source link

Should mlr3 be in Depends instead of Imports? #20

Closed ledell closed 3 years ago

ledell commented 3 years ago

Currently, all the packages that mlr3automl depends on are in the Imports section of the DESCRIPTION file. I wonder if it would be helpful to move mlr3 to Depends, so you don't need to explicitly load it when running the examples?

> library(mlr3automl)
> iris_task <- tsk('iris')
Error in tsk("iris") : could not find function "tsk"

You have to do this:

library(mlr3)
library(mlr3automl)

iris_task <- tsk('iris')
model <- AutoML(iris_task)
model$train()
a-hanf commented 3 years ago

Thanks for the suggestion and sorry for leaving this unanswered for so long. We discussed this internally and won't be making changes here