Open nabenabe0928 opened 1 year ago
I guess the surrogate is fine with out of domain values but i agree that an out of domain check would be helpful. I ended up creating my own configs with a validate()
and manually specifying the bounds to counter this.
This is a very valid point, thank you for bringing it up! The way the number of epochs is currently being handled is definitely sub-optimal and will be revised in the upcoming updates.
Regarding the domain checks, this is something that requires some discussion.
ConfigSpace
implementation of the search space is enough to provide the requisite information to any algorithms that need the domain bounds, but maybe this can be improved.What is your opinion?
Hi, thanks for the response:)
For the tabular, I don't think we need the check, but for the surrogate model, it would be grateful to have warning at least because XGBoost gives, by nature of the decision tree, the same prediction on each parameter which is rounded to the original range.
This is not exactly an issue but leads to misunderstandings, so I would like to mention it here.
nepochs
in the argument ofBenchmark.__call__
orepoch
inconfig
dict? (I know the answer isnepochs
, but it is ambiguous until we run)N=100
) and it seems fidelity parameters are rounded somehow?I would really appreciate if we can choose a mode, either clipping automatically or raising errors anytime you get invalid items.