Skip to content
This repository was archived by the owner on Feb 28, 2024. It is now read-only.

[MRG] add ThresholdStopper() #1000

Merged
glouppe merged 6 commits into from
Apr 11, 2021
Merged

[MRG] add ThresholdStopper() #1000

glouppe merged 6 commits into from
Apr 11, 2021

Conversation

@GuillaumeSimo
Copy link
Contributor

@GuillaumeSimo GuillaumeSimo commented Feb 11, 2021 •
edited by kernc
Loading

@pep8speaks
Copy link

pep8speaks commented Feb 11, 2021 •
edited
Loading

Hello @GuillaumeSimo! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-02-12 08:56:10 UTC

@GuillaumeSimo GuillaumeSimo changed the title add ThresholdStopper() [MRG] add ThresholdStopper() Feb 11, 2021
kernc
kernc suggested changes Feb 12, 2021
View reviewed changes
@GuillaumeSimo GuillaumeSimo requested a review from kernc February 12, 2021 08:54
@glouppe glouppe merged commit d6c6e07 into scikit-optimize:master Apr 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@kernc kernc kernc approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

EarlyStopper callback given a threshold

4 participants

@GuillaumeSimo @pep8speaks @kernc @glouppe