-
Notifications
You must be signed in to change notification settings - Fork 15
Cod 732 support perf profiling in python walltime #79
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
Cod 732 support perf profiling in python walltime #79
Conversation
9d3b8cb to
e6c7442
Compare
e6c7442 to
a3717e4
Compare
da53257 to
5330f3f
Compare
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.
Exciting, curious to see if there is any performance changes, especially on the instrumentation results. Can you fix the CI to see the results?
5330f3f to
c2a1644
Compare
cb73581 to
650a49c
Compare
art049
left a comment
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.
not much left, we're getting close :)
648dc66 to
3bf2db5
Compare
|
Currently still failing due to CodSpeedHQ/instrument-hooks#2, but should be good to go once merged. Lmk if the requested changes look good. |
This comment was marked as outdated.
This comment was marked as outdated.
CodSpeed Instrumentation Performance ReportMerging #79 will degrade performances by 3.86%Comparing Summary
Benchmarks breakdown
|
a827702 to
2a2a7de
Compare
b034178 to
3302632
Compare
3302632 to
7d9c538
Compare
174c86a to
467118c
Compare
467118c to
729e66a
Compare
729e66a to
7583000
Compare
Depends on: CodSpeedHQ/instrument-hooks#1 (must be merged and released before merging this)