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
StringCatHyperParameter, IntCatHyperParameter, FloatCatHyperParameter are not needed. Just CatHyperParameter.
ParamTypes is also unneeded. String is a scalar, scalar is a degenerate cathegory. Distributions should be decoupled from types, for distributions scipy.stats should be used, and for types ... types themselves.
Hi @KOLANICH hyperparameters will soon undergo a design review and refactoring in order to support several of the new requirements, like #80, #111, #112 and #113
So we take notes of your suggestion as well, but this stays closed for now, as it won't be implemented as something on its own.
IMHO:
StringCatHyperParameter
,IntCatHyperParameter
,FloatCatHyperParameter
are not needed. JustCatHyperParameter
.ParamTypes
is also unneeded. String is a scalar, scalar is a degenerate cathegory. Distributions should be decoupled from types, for distributionsscipy.stats
should be used, and for types ... types themselves.You may want to look at my implementation
The text was updated successfully, but these errors were encountered: