Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convenience Method for Adding Previous Runs? #805

Closed
benjamc opened this issue Jan 6, 2022 · 5 comments · Fixed by #875
Closed

Convenience Method for Adding Previous Runs? #805

benjamc opened this issue Jan 6, 2022 · 5 comments · Fixed by #875
Labels

Comments

@benjamc
Copy link
Contributor

benjamc commented Jan 6, 2022

Restoring SMAC takes some lines of code. What do you think, @renesass @dengdifan @timruhkopf , should we also add a convenience method?
E.g. Optuna allows to add previous trials with an add_trial() method.

@benjamc benjamc added the feature label Jan 6, 2022
@mlindauer
Copy link
Contributor

I think that makes a lot of sense. It will make it easier to compare different optimizers with each other if we can block the initial design.

@mfeurer
Copy link
Contributor

mfeurer commented Jan 10, 2022

Would this be 1/2 of a suggest&report interface?

@stale
Copy link

stale bot commented Jun 17, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 17, 2022
@renesass renesass removed the stale label Jun 23, 2022
@renesass renesass linked a pull request Aug 18, 2022 that will close this issue
@renesass
Copy link
Collaborator

I added the option to pass a runhistory in development 2.0. The initial configs and incumbent are derived from the runhistory directly. Does that fix the problem?

@mfeurer
Copy link
Contributor

mfeurer commented Aug 18, 2022

Passing a runhistory still requires to know about runhistories. A function add_trial() or similar would make it easy for the user to just write a for-loop and add results one by one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants