-
Notifications
You must be signed in to change notification settings - Fork 320
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
Call "get_improvement_over_baseline" when making scheduler log #2050
Conversation
This pull request was exported from Phabricator. Differential Revision: D51858443 |
81dd5fd
to
1cba6a7
Compare
…ook#2050) Summary: Integrating SchedulerCompletedRecord.from_scheduler with `scheduler.get_improvement_over_baseline` Differential Revision: D51858443
This pull request was exported from Phabricator. Differential Revision: D51858443 |
1cba6a7
to
eaf8bf3
Compare
…ook#2050) Summary: Integrating SchedulerCompletedRecord.from_scheduler with `scheduler.get_improvement_over_baseline` Differential Revision: D51858443
This pull request was exported from Phabricator. Differential Revision: D51858443 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2050 +/- ##
=======================================
Coverage 94.52% 94.52%
=======================================
Files 460 460
Lines 44625 44638 +13
=======================================
+ Hits 42182 42195 +13
Misses 2443 2443 ☔ View full report in Codecov by Sentry. |
…ook#2050) Summary: Integrating SchedulerCompletedRecord.from_scheduler with `scheduler.get_improvement_over_baseline` Reviewed By: mpolson64 Differential Revision: D51858443
eaf8bf3
to
90c314f
Compare
This pull request was exported from Phabricator. Differential Revision: D51858443 |
90c314f
to
2f0761d
Compare
…ook#2050) Summary: Integrating SchedulerCompletedRecord.from_scheduler with `scheduler.get_improvement_over_baseline` Reviewed By: mpolson64 Differential Revision: D51858443
This pull request was exported from Phabricator. Differential Revision: D51858443 |
…ook#2050) Summary: Integrating SchedulerCompletedRecord.from_scheduler with `scheduler.get_improvement_over_baseline` Reviewed By: mpolson64 Differential Revision: D51858443
2f0761d
to
64ce51a
Compare
…ook#2050) Summary: Integrating SchedulerCompletedRecord.from_scheduler with `scheduler.get_improvement_over_baseline` Reviewed By: mpolson64 Differential Revision: D51858443
64ce51a
to
9fe7226
Compare
This pull request was exported from Phabricator. Differential Revision: D51858443 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D51858443 |
This pull request has been merged in 46e61d1. |
Summary: Integrating SchedulerCompletedRecord.from_scheduler with
scheduler.get_improvement_over_baseline
Differential Revision: D51858443