AORTA-23 : Phase 4 : Generate excel report for baseline vs test configuration#101
Conversation
prosenjitdhole
changed the title
There was a problem hiding this comment.
Pull request overview
Adds Phase 4 support for generating an Excel report that compares baseline vs test HW queue sweep results across multiple workloads (Mode C), integrating it into the existing reporting pipeline.
Changes:
- Wire comparison-mode pipeline step to generate a multi-workload comparison Excel workbook.
- Implement Mode C Excel generation: summary, by-stream throughput/latency, regressions/improvements, missing workloads, and environment sheets + conditional formatting.
- Export
generate_comparison_excelvia the generators package.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
src/aorta/report/pipelines/hwqueue_pipeline.py |
Calls the new comparison Excel generator during Mode C execution and records generated outputs + regression/improvement lists. |
src/aorta/report/generators/hwqueue_excel.py |
Implements generate_comparison_excel and helpers to build/format the comparison workbook. |
src/aorta/report/generators/__init__.py |
Re-exports generate_comparison_excel. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@prosenjitdhole I've opened a new pull request, #131, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@prosenjitdhole I've opened a new pull request, #132, to work on those changes. Once the pull request is ready, I'll request review from you. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@prosenjitdhole I've opened a new pull request, #133, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@prosenjitdhole I've opened a new pull request, #134, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…parison sheet (#133) * Initial plan * Add TODO noting baseline and test labels are assumed to be distinct Co-authored-by: prosenjitdhole <239307697+prosenjitdhole@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: prosenjitdhole <239307697+prosenjitdhole@users.noreply.github.com>
* Initial plan * Fix zero-value handling: use `is not None` instead of truthiness checks Co-authored-by: prosenjitdhole <239307697+prosenjitdhole@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: prosenjitdhole <239307697+prosenjitdhole@users.noreply.github.com> Co-authored-by: prosenjitdhole <prosenjit.dhole@amd.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
* Initial plan * Add TODO comments for is not None checks in _build_throughput_by_streams_sheet Co-authored-by: prosenjitdhole <239307697+prosenjitdhole@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: prosenjitdhole <239307697+prosenjitdhole@users.noreply.github.com> Co-authored-by: prosenjitdhole <prosenjit.dhole@amd.com>
No description provided.