automl / fanova

Functional ANOVA
122 stars 45 forks source link

FIX #96 by removing swap (also add support for Ordinals and simplify code) #97

Closed shukon closed 4 years ago

shukon commented 4 years ago

Most of it is just swapping the logic (asking isinstance(parameter, NumericalHyperparameter) instead of isinstance(parameter, CategoricalHyperparameter) and making the code more maintainable.

shukon commented 4 years ago

Good enough to release?

AndreBiedenkapp commented 4 years ago

I think so yes.