You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently there are many functions taking in user_priors: bool, SuccessiveHalvinh/Hyperband + Sampling policy takes in user_priors: bool and BayesianOptimizer's use disable_priors.
We should just have one, use_priors: bool as it leads to inconsistencies in yaml files and definitions of various objects, even worse, one is a double negative, i.e. disable_priors = False which implies use_priors = True.
Currently there are many functions taking in
user_priors: bool
, SuccessiveHalvinh/Hyperband + Sampling policy takes inuser_priors: bool
and BayesianOptimizer's usedisable_priors
.We should just have one,
use_priors: bool
as it leads to inconsistencies in yaml files and definitions of various objects, even worse, one is a double negative, i.e.disable_priors = False
which impliesuse_priors = True
.@Neeratyoy
The text was updated successfully, but these errors were encountered: