-
Notifications
You must be signed in to change notification settings - Fork 400
add codspeed benchmark runner #4394
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
Conversation
d08b0ad
to
ec346f1
Compare
0ff07e7
to
a32f05a
Compare
d61f07f
to
f841c2f
Compare
CodSpeed Performance ReportCongrats! CodSpeed is installed 🎉
You will start to see performance impacts in the reports once the benchmarks are run from your default branch.
|
These numbers are likely too small to be used for performance comparisons, I think aiming for 5-10ms per benchmark test is likely a good starting point, microseconds benchmarks likely need more iterations |
Agreed. I think we should make an update to all benchmarks after this pull-request lands. Most of those benchmarks are not needed, but I'm pretty sure it's up to the discussion. |
ba9a38d
to
c8e1d1b
Compare
c8e1d1b
to
6699527
Compare
688ed66
to
58a6ce9
Compare
do not build bench-tools on windows use local repository for google benchmark revert ci.bazelrc changes
58a6ce9
to
b09b0b3
Compare
Completes the work started by @art049 at #4137