-
Notifications
You must be signed in to change notification settings - Fork 2
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
Include times per line in statistics #830
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #830 +/- ##
==========================================
+ Coverage 71.84% 71.92% +0.08%
==========================================
Files 202 202
Lines 7274 7296 +22
Branches 1125 1227 +102
==========================================
+ Hits 5226 5248 +22
- Misses 1691 1694 +3
+ Partials 357 354 -3 ☔ View full report in Codecov by Sentry. |
TODO: standard deviation for 1 run is 0 because it gets summarized from just one run, and then the standard deviation for the total is also 0 because it gets calculated based on the other standard deviations. that's wrong & bad This and a lot of other problems will be solved by flipping summarization order (currently we summarize all runs of a file first, then summarize over those. we should be summarizing all files of the nth run, and then summarizing all runs) |
This pull request is included in v2.0.3 (see Release v2.0.3 (Fine-Grained Benchmarks)). |
1 similar comment
This pull request is included in v2.0.3 (see Release v2.0.3 (Fine-Grained Benchmarks)). |
No description provided.