coseal / aslib-r Goto Github PK

probably some brackets are mismatched

The instance names are sorted a certain way in the ASLIB-files. Depending on locale the sorting order on other computers can be different. The scenario-names as contained in algo.runstatus get resorted, whereas those in algo.runs do not get resorted. Therefore these two orderings can become inconsistent. Note that I have not encountered a bug as a direct result form this inconsistency, I simply thought it was worth pointing out.

Proposed fix: resort algo.runs as well.
Note: all other datastructures in an aslib scenario containing instance-names should also be checked for consistency

Reproduceable example

library(aslib)
scenarioFolderLocation = "C:...\aslib_data-aslib-v2.0"
scenarioName = "SAT11-HAND"
scenario = parseASScenario(paste(scenarioFolderLocation, scenarioName, sep=""))
print(scenario$algo.runstatus$instance_id[1]) #[1] "./SAT02/handmade/chu-min-li/urquhart/urquhart3_25bis.shuffled.cnf"

print(scenario$algo.runs$instance_id[1]) #"./SAT11/crafted/kullmann/VanDerWaerden/VanderWaerden_pd_3k/VanDerWaerden_pd_2-3-21_399.cnf"

with session info (I guess you might have to change your locale to mine to reproduce it)

sessionInfo()
R version 3.2.3 (2015-12-10)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 7 x64 (build 7601) Service Pack 1

locale:
[1] LC_COLLATE=Dutch_Belgium.1252 LC_CTYPE=Dutch_Belgium.1252 LC_MONETARY=Dutch_Belgium.1252 LC_NUMERIC=C
[5] LC_TIME=Dutch_Belgium.1252

attached base packages:
[1] stats graphics grDevices utils datasets methods base

other attached packages:
[1] aslib_0.1

loaded via a namespace (and not attached):
[1] parallelMap_1.3 Rcpp_0.12.7 plyr_1.8.4 base64enc_0.1-3 BatchJobs_1.6
[6] tools_3.2.3 RWekajars_3.9.0-1 digest_0.6.10 corrplot_0.77 RSQLite_1.0.0
[11] tibble_1.2 gtable_0.2.0 checkmate_1.8.1 shiny_0.14 DBI_0.5-1
[16] yaml_2.1.13 parallel_3.2.3 rJava_0.9-8 stringr_1.1.0 dplyr_0.5.0
[21] ggvis_0.4.3 grid_3.2.3 data.table_1.9.6 R6_2.1.3 survival_2.38-3
[26] fail_1.3 RWeka_0.4-29 llama_0.9.1 reshape2_1.4.1 ggplot2_2.1.0
[31] mlr_2.9 sendmailR_1.2-1 magrittr_1.5 splines_3.2.3 backports_1.0.3
[36] scales_0.4.0 BBmisc_1.10 ParamHelpers_1.9 htmltools_0.3.5 BatchExperiments_1.4.1
[41] assertthat_0.1 xtable_1.8-2 mime_0.5 colorspace_1.2-6 httpuv_1.3.3
[46] brew_1.0-6 stringi_1.1.1 munsell_0.4.3 chron_2.3-47

for measure type = "quality" we always impute with 10 * max_val.

a) it is bad that the user cannot change this behavior / magic constant. what happens here should be exposed. or at least documented and the user told to imputed himself.

b) it does not work at all if we maximize. bug

Hi,

I was trying to do algorithm selection for machine learning data. When building a model the R interpreter makes an error. I pasted my commands as well as the output:

feurerm@aadpool4:~/Software/R-3.2.1/bin$ ./R

R version 3.2.1 (2015-06-18) -- "World-Famous Astronaut"
Copyright (C) 2015 The R Foundation for Statistical Computing
Platform: x86_64-unknown-linux-gnu (64-bit)

R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.

  Natural language support but running in an English locale

R is a collaborative project with many contributors.
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.

Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.

> require(llama)
Loading required package: llama
Loading required package: mlr
Loading required package: BBmisc
Loading required package: ggplot2
Loading required package: ParamHelpers

Attaching package: ‘llama’

The following object is masked from ‘package:BBmisc’:

    normalize

> library(aslib)
> scenario = parseASScenario("/home/feurerm/mhome/projects/2015_AutoSklearn_vs_AutoWEKA/metadata/aslib_matrix/aslib/binary.classification_dense_bac_metric")
Warning in parseASScenario("/home/feurerm/mhome/projects/2015_AutoSklearn_vs_AutoWEKA/metadata/aslib_matrix/aslib/binary.classification_dense_bac_metric") :
  No cv file exists for scenario at:
/home/feurerm/mhome/projects/2015_AutoSklearn_vs_AutoWEKA/metadata/aslib_matrix/aslib/binary.classification_dense_bac_metric
> ldf = convertToLlama(scenario)
> folds = cvFolds(ldf)
> model = classify(makeLearner("classif.J48"), folds)
Error in `[.data.frame`(data$data[data$train[[i]], ], data$features) : 
  undefined columns selected
> 

The R interpreter is freshly compiled and runs under Ubuntu 14.04. I tried the ASP benchmark from the ASLIB and it worked without problems.

Tests are currently excluded via .Rbuildignore. Including them causes errors because of the path to the test data sets under inst/.

Replace with smaller dataset(s) and make sure that they run without warnings.

http://coseal.github.io/aslib-r/scenario-pages/SAT11-RAND/llama.htm
"The feature steps correspond to the following 115 / 115 features:"

These are not the default features we agreed on.
Bug on Homepage or in aslib-r/llama?