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

Add OptunaSearchCV with terminator #225

Merged
merged 2 commits into from
Nov 20, 2023

Conversation

Alnusjaponica
Copy link
Collaborator

Motivation

Thanks to optuna/optuna#5098, now we can use terminator in OptunaSearchCV. This example shows how to use it.

Description of the changes

The example itself is basically the same as sklearn/sklearn_optuna_search_cv_simple.py except that TerminatorCallback with default setting is passed to OptunaSearchCV.
I added the example under terminator/ so that we don't have to change dependency of each module.

@Alnusjaponica Alnusjaponica force-pushed the add-terminator-search-cv branch from a5304ce to fc17336 Compare November 9, 2023 03:57
@Alnusjaponica Alnusjaponica force-pushed the add-terminator-search-cv branch from fc17336 to 6194ac9 Compare November 9, 2023 03:59
@Alnusjaponica Alnusjaponica marked this pull request as ready for review November 9, 2023 07:56
@c-bata
Copy link
Member

c-bata commented Nov 14, 2023

@nabenabe0928 Could you review this PR?

@c-bata
Copy link
Member

c-bata commented Nov 14, 2023

@Alnusjaponica Could you update README.md to add a link?
https://github.com/optuna/optuna-examples#examples-of-terminator

@nabenabe0928
Copy link
Collaborator

@nabenabe0928 Could you review this PR?

Yes

@Alnusjaponica
Copy link
Collaborator Author

@c-bata I updated README.md. PTAL.

@c-bata
Copy link
Member

c-bata commented Nov 16, 2023

@Alnusjaponica Thank you for the update!

@nabenabe0928 Could you please let me know once you approve? I'll then merge this PR.

Copy link
Collaborator

@nabenabe0928 nabenabe0928 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR, LGTM!

@HideakiImamura HideakiImamura merged commit 5d6e19a into optuna:main Nov 20, 2023
6 checks passed
@HideakiImamura HideakiImamura added this to the v3.5.0 milestone Nov 20, 2023
@Alnusjaponica Alnusjaponica deleted the add-terminator-search-cv branch November 20, 2023 08:53
@nabenabe0928 nabenabe0928 added the document Documentation related. label Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document Documentation related.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants