automl / amltk

A build-it-yourself AutoML Framework
https://automl.github.io/amltk/
BSD 3-Clause "New" or "Revised" License
68 stars 6 forks source link

fix(CVEvaluator): `clone` the estimator before use #249

Closed eddiebergman closed 10 months ago

eddiebergman commented 10 months ago

Whoops, somehow the estimator wasn't getting cloned before being CVFitted on. This wasn't caught by the tests as the default model we used could be warm-started and refit without issue. I ran across this bug while using this on an MLPClassifier.