[Internal API] `user_priors`, `use_priors`, `disable_priors`

automl / neps

Neural Pipeline Search (NePS): Helps deep learning experts find the best neural pipeline.

https://automl.github.io/neps/

Apache License 2.0

39 stars 11 forks source link

[Internal API] `user_priors`, `use_priors`, `disable_priors` #110

Open eddiebergman opened 1 week ago

eddiebergman commented 1 week ago

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.

@Neeratyoy

Neeratyoy commented 1 week ago

I agree.

There are 2 ways to tackle this:

Either this is its own push that is handled along with the #28, or
This is taken care of part of the bigger BO structure refactor in plan

eddiebergman commented 1 week ago

I would vote for the first way, before the second one