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

[Optimizers] Make all available optimizers through a "string" be instatiable without kwargs #118

Open
3 tasks
eddiebergman opened this issue Jul 30, 2024 · 0 comments
Labels
ux Something that would improve user experience

Comments

@eddiebergman
Copy link
Contributor

eddiebergman commented Jul 30, 2024

WIP Issue... Just submitting it to get a number to put in code for now


The optimizers support a lot of configuration but we make sure to have sensible defaults. However some optimizers do not run without extra keyword arguments specified

  • All optimizers should be able to be instantiated with OptimizerCls(pipeline_space=space)
    • "assisted_regularized_evolution": assert not assisted or (assisted and assisted_zero_cost_proxy is not None) - It won't run with defaults, it has assisted as True and assisted_zero_cost_proxy = None, needing to be specified.
    • "successive_halving_prior" requires a budget to be specified while "successive_halving"does not. I'm not sure which one is incorrect. It seems likeSuccessiveHalvingwill just passbudget=None` to base Optimizer and move on with its day... not sure what the behaviour is in this case.
@eddiebergman eddiebergman added this to the Optimizer Overhaul milestone Jul 30, 2024
@eddiebergman eddiebergman added optim ux Something that would improve user experience and removed optim labels Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux Something that would improve user experience
Projects
Status: No status
Development

No branches or pull requests

1 participant