-
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
Add evaluate_oracle
method to BenchmarkRunner
#2705
Conversation
This pull request was exported from Phabricator. Differential Revision: D61431980 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2705 +/- ##
=======================================
Coverage 95.28% 95.28%
=======================================
Files 493 493
Lines 47899 47903 +4
=======================================
+ Hits 45639 45643 +4
Misses 2260 2260 ☔ View full report in Codecov by Sentry. |
This pull request was exported from Phabricator. Differential Revision: D61431980 |
Summary: Pull Request resolved: facebook#2705 Docstring should make this self-explanatory. In addition to the increased ease of development, enables reaping a lot of LOC in D61415525 Reviewed By: Balandat Differential Revision: D61431980
3cd6546
to
81cc983
Compare
Summary: Pull Request resolved: facebook#2705 Docstring should make this self-explanatory. In addition to the increased ease of development, enables reaping a lot of LOC in D61415525 Reviewed By: Balandat Differential Revision: D61431980
81cc983
to
ea82e45
Compare
This pull request was exported from Phabricator. Differential Revision: D61431980 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D61431980 |
Summary: Pull Request resolved: facebook#2705 Docstring should make this self-explanatory. In addition to the increased ease of development, enables reaping a lot of LOC in D61415525 Reviewed By: Balandat Differential Revision: D61431980
ea82e45
to
a244d6a
Compare
This pull request was exported from Phabricator. Differential Revision: D61431980 |
Summary: Pull Request resolved: facebook#2705 Docstring should make this self-explanatory. In addition to the increased ease of development, enables reaping a lot of LOC in D61415525 Reviewed By: Balandat Differential Revision: D61431980
a244d6a
to
df42735
Compare
This pull request was exported from Phabricator. Differential Revision: D61431980 |
Summary: Pull Request resolved: facebook#2705 Docstring should make this self-explanatory. In addition to the increased ease of development, enables reaping a lot of LOC in D61415525 Reviewed By: Balandat Differential Revision: D61431980
df42735
to
b031fd1
Compare
Summary: Pull Request resolved: facebook#2705 Docstring should make this self-explanatory. In addition to the increased ease of development, enables reaping a lot of LOC in D61415525 Reviewed By: Balandat Differential Revision: D61431980
This pull request was exported from Phabricator. Differential Revision: D61431980 |
b031fd1
to
e0841db
Compare
This pull request has been merged in 36194d3. |
Summary:
Docstring should make this self-explanatory.
In addition to the increased ease of development, enables reaping a lot of LOC in D61415525
Reviewed By: Balandat
Differential Revision: D61431980