-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
parse-localbenchmarks: separate standard deviation #16812
parse-localbenchmarks: separate standard deviation #16812
Conversation
Go benchmark results include an Average, represented as <mean> ± <standard deviation> This is suboptimal for many reasons: * Some web server somewhere in our CI pipeline (Cirrus? Google? Gitlab? I have no idea) sends the wrong mime-type header, rendering the CSV weird-looking in a browser. Not that it's intended for a browser, but we have to debug/verify manually once in a while. * The spaces and +/- makes it less machine-readable. Solution: split the "Average" field into two: Average, and Standard Deviation. And, as a courtesy to human readers, add a new column with SD as a percentage. Signed-off-by: Ed Santiago <santiago@redhat.com>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: edsantiago The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/lgtm |
/hold cancel |
Go benchmark results include an Average, represented as
This is suboptimal for many reasons:
Some web server somewhere in our CI pipeline (Cirrus?
Google? Gitlab? I have no idea) sends the wrong mime-type
header, rendering the CSV weird-looking in a browser.
Not that it's intended for a browser, but we have to
debug/verify manually once in a while.
The spaces and +/- makes it less machine-readable.
Solution: split the "Average" field into two: Average, and
Standard Deviation. And, as a courtesy to human readers,
add a new column with SD as a percentage.
Signed-off-by: Ed Santiago santiago@redhat.com