-
Notifications
You must be signed in to change notification settings - Fork 303
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
Remove unnecessary wrapper-helper around early-stopping call #2666
Conversation
This pull request was exported from Phabricator. Differential Revision: D55710758 |
This pull request was exported from Phabricator. Differential Revision: D55710758 |
69bdc4b
to
7c6e286
Compare
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Differential Revision: D55710758
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Differential Revision: D55710758
7c6e286
to
3d00eb2
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Differential Revision: D55710758
3d00eb2
to
3cab75a
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Differential Revision: D55710758
3cab75a
to
5bfdb10
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Differential Revision: D55710758
5bfdb10
to
7e36810
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Differential Revision: D55710758
7e36810
to
df9f59b
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Reviewed By: saitcakmak, Balandat Differential Revision: D55710758
df9f59b
to
c340084
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Reviewed By: saitcakmak, Balandat Differential Revision: D55710758
c340084
to
007c4ef
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Reviewed By: saitcakmak, Balandat Differential Revision: D55710758
007c4ef
to
eaf5675
Compare
This pull request was exported from Phabricator. Differential Revision: D55710758 |
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Reviewed By: saitcakmak, Balandat Differential Revision: D55710758
eaf5675
to
72371c6
Compare
…k#2666) Summary: Pull Request resolved: facebook#2666 As titled Reviewed By: saitcakmak, Balandat Differential Revision: D55710758
This pull request was exported from Phabricator. Differential Revision: D55710758 |
72371c6
to
2f1d3fc
Compare
This pull request has been merged in a4a97d6. |
…at was timing out the CI Summary: The `EarlyStopsInsteadOfNormalCompletionScheduler` test class was overwriting `should_stop_trials_early`, which was being used to make the early stopping decisions prior to facebook#2666. After that diff, the overwrite became a no-op, leading to this test hanging out and failing the CI. Reviewed By: danielcohenlive, mpolson64 Differential Revision: D62784565
…at was timing out the CI (#2770) Summary: Pull Request resolved: #2770 The `EarlyStopsInsteadOfNormalCompletionScheduler` test class was overwriting `should_stop_trials_early`, which was being used to make the early stopping decisions prior to #2666. After that diff, the overwrite became a no-op, leading to this test hanging out and failing the CI. Reviewed By: danielcohenlive, mpolson64 Differential Revision: D62784565 fbshipit-source-id: 11e3b6be0057e82afd82b706aeda54af686a7c8b
Summary: As titled
Differential Revision: D55710758