automl / neps

Neural Pipeline Search (NePS): Helps deep learning experts find the best neural pipeline.
https://automl.github.io/neps/
Apache License 2.0
61 stars 13 forks source link

[Optimizers] Multifidelity algorithms should check for `min_budget == 0` #123

Open eddiebergman opened 4 months ago

eddiebergman commented 4 months ago

Right now, we get a ZeroDivisionError with SuccessiveHalving based algorithms because the fidelity parameter has a lower bound of 0 in a testing setup.

Moreover, perhaps we should explicitly check that it's positive all the time? This might belong in the search space instead though.

https://github.com/automl/neps/blob/82b48903d0896db7d84ea2810f567a29d01688c3/neps/optimizers/multi_fidelity/successive_halving.py#L121-L123