-
Notifications
You must be signed in to change notification settings - Fork 25.5k
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
Enhanced benchmark.py for improved metrics handling and multi-commit … #31692
base: main
Are you sure you want to change the base?
Conversation
Hi @54J4N, thanks for opening a PR! Could you provide some more context for these changes? Is there a discussion on the forums or related issue describing the motivations? |
Hi Amy, |
@54J4N It would help us review and decide if this should be an addition to the codebase if you could provide some more information:
How does this change the metrics handling?
Could you provide some more context here? What does 'multi-commit' becnhmarking mean here? Could you provide code snippets showing how this can and should be done?
This is best show by providing an example command and desired output |
Enhanced Metrics Handling -Integrated new metrics to capture detailed performance data:
Impact:
Support Multi-Commit Benchmarking
Context:
Providing Structured JSON Reports
|
Changes: