automl / amltk

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

fix(pipeline): Ensure optimizer is updated with report #261

Closed eddiebergman closed 4 months ago

eddiebergman commented 4 months ago

Fixes a bug where using pipeline.optimize was not actually calling the tell() of the optimizer. Added a test to ensure this does not get removed.

@LennartPurucker tagged for review, no need to if you don't have time as I unfortunately need to merge this now. I've confirmed it works with the tests but also in the other setting via good ol' print debugging. Changes are annotated to expedite the process.