Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation fails with current MLR version #7

Open
tbrieden opened this issue Jul 30, 2018 · 1 comment
Open

Installation fails with current MLR version #7

tbrieden opened this issue Jul 30, 2018 · 1 comment
Assignees

Comments

@tbrieden
Copy link

As described in the installation section, the installation of automlr is currently only possible by using the adapted MLR version that contains a specific CPO extension. However, this version of MLR is not up-to-date and the pull request mentioned in the installation section is already merged into MLR. Therefore, I was trying to install automlr with the current MLR version(2.12.1). This fails with the following error message:

roxygenise()
First time using roxygen2. Upgrading automatically...
Updating roxygen version in /tmp/RtmpYSmN2Q/devtoolsf733f14c882/mlr-org-automlr-8e74e08/DESCRIPTION
Writing NAMESPACE
Loading automlr
Loading required package: mlr
Loading required package: ParamHelpers
Error in makeCPO("cfimputer", .datasplit = "factor", .properties.adding = "missings", (from CPO.R#24) :
could not find function "makeCPO"

It suspect that the makeCPO method was removed again from MLR and moved to mlrCPO. Therefore, I installed mlrCPO and added it as a dependency to a local clone of the automlr repo. Unfortunately, the installations fails again, giving me the following error:

roxygenise()
First time using roxygen2. Upgrading automatically...
Updating roxygen version in /tmp/RtmpYSmN2Q/filef735918e0f6/automlr/DESCRIPTION
Writing NAMESPACE
Loading automlr
Loading required package: mlr
Loading required package: ParamHelpers
Loading required package: mlrCPO
Error in makeCPO("cfimputer", .datasplit = "factor", .properties.adding = "missings", (from CPO.R#24) :
unused arguments (.datasplit = "factor", .properties.adding = "missings", .stateless = TRUE, cpo.trafo = {
for (n in names(data)) {
data[[n]] = factor(data[[n]], levels = c("MISSING", levels(data[[n]])))
data[[n]][is.na(data[[n]])] = "MISSING"
}
data
})

It seems that the CPO calls used in master/R/CPO.R are outdated. It there any chance to get automlr running with the current MLR release or would this require a lot of work on automlr to get this up-to-date?

@tbrieden
Copy link
Author

tbrieden commented Aug 6, 2018

I switched to using mlrMBO directly instead of automlr. Therefore, the questions I asked here are not that important to me anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants