-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat(sweeps): optuna supports multi-objective optimization #28
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple more questions
if not config.get("sweep_args", {}).get("sweep_id"): | ||
# not a sweep, just finish the run and return | ||
if not args or not args.get("sweep_id"): | ||
# when the config has no sweep args, this is being run directly from CLI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the user is running the wandb launch -j <job>
here? Is this specifically done to create the job?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
now this would be triggered by running python optuna_scheduler.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And a user would do that to create the job. Sg
" does not match number of metrics defined " | ||
f"({self._metric_defs})" | ||
f"({self._metric_defs}). Specify metrics for optimization" | ||
" in the scheduler.settings.metrics portion of the sweep config" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What would a user have done to create this case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im not sure its possible, i included this error message in case. If we had a testing suite.... I'll play around with it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥
https://wandb.atlassian.net/browse/WB-14249