search

LMZimmer / Auto-PyTorch_refactor

Apache License 2.0
0 stars 2 forks source link

Ensemble builder #36

Closed franchuterivera closed 3 years ago

franchuterivera commented 3 years ago

Initial Commit with an empty file, mainly to test the check-api

ravinkohli commented 3 years ago

Should we wait for the dataset class to be merged and then make changes to this PR before merging it? Otherwise, for me the changes are good.

franchuterivera commented 3 years ago

We need the changes in this PR for SMAC. It also fixes the backend

When building an ensemble we use smac output, so we do not care about the data manager.

Nevertheless, I am reading the datamanager to make any time test performance, but is a 1 line change when the dataset/datamanager is ready

ravinkohli commented 3 years ago

We need the changes in this PR for SMAC. It also fixes the backend

When building an ensemble we use SMAC output, so we do not care about the data manager.

Nevertheless, I am reading the data manager to make any time test performance, but is a 1 line change when the dataset/data manager is ready

Okay then, from my side it can be merged. Let's wait for Lucas to review it