mlr-org / paradox

ParamHelpers Next Generation
https://paradox.mlr-org.com
GNU Lesser General Public License v3.0
28 stars 7 forks source link

Bad interactions ParamHelpers if loaded at the same time #349

Closed bthieurmel closed 3 years ago

bthieurmel commented 3 years ago

Retrieving error with ParamDbl, ParamInt, ... and so an empty ParamSet if ParamHelpers is loaded in the session :

library("mlr3tuning")

# ok
tune_ps = ParamSet$new(list(
  ParamDbl$new("cp", lower = 0.001, upper = 0.1),
  ParamInt$new("minsplit", lower = 1, upper = 10)
))

ParamDbl$new("cp", lower = 0.001, upper = 0.1)
tune_ps 

library("ParamHelpers")
# error
tune_ps_empty = ParamSet$new(list(
  ParamDbl$new("cp", lower = 0.001, upper = 0.1),
  ParamInt$new("minsplit", lower = 1, upper = 10)
))
ParamDbl$new("cp", lower = 0.001, upper = 0.1)
tune_ps_empty 
R version 4.0.5 (2021-03-31)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 20.04.2 LTS

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.9.0
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.9.0

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8    
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C                 
 [9] LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C       

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

other attached packages:
[1] ParamHelpers_1.14 mlr3tuning_0.8.0  paradox_0.7.1     mlr3_0.11.0      

loaded via a namespace (and not attached):
 [1] Rcpp_1.0.6           lgr_0.4.2            mlr3misc_0.9.1       palmerpenguins_0.1.0 crayon_1.4.1        
 [6] parallelly_1.24.0    R6_2.5.0             backports_1.2.1      uuid_0.1-4           data.table_1.14.0   
[11] rstudioapi_0.13      bbotk_0.3.2          checkmate_2.0.0      fastmatch_1.1-0      fst_0.9.4           
[16] tools_4.0.5          parallel_4.0.5       compiler_4.0.5       BBmisc_1.11    
mllg commented 3 years ago

I've added a warning which gets triggered if both packages are loaded.