-
Notifications
You must be signed in to change notification settings - Fork 929
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
Run benchmarks with --profile profiling
#5927
Conversation
Looks like |
582a0bf
to
1fc293b
Compare
1fc293b
to
317306f
Compare
CodSpeed Performance ReportMerging #5927 will degrade performances by 13.79%Comparing Summary
Benchmarks breakdown
|
The regressions are because the |
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.
I think it makes sense to merge with the regressions since we are changing how we measure perf with this PR.
Summary
The CodSpeed flamegraphs are currently useless after #5745.