AORTA-23 : Phase 3 : Generating excel report for single and sweep run#100
AORTA-23 : Phase 3 : Generating excel report for single and sweep run#100prosenjitdhole wants to merge 5 commits intoprosenj_cli_hq_eval_report_phase_2from
Conversation
prosenjitdhole
changed the title
prosenjitdhole
changed the title
There was a problem hiding this comment.
Pull request overview
Implements Phase 3 of the HW Queue Eval report pipeline by adding Excel report generation for single-run (Mode A) and sweep (Mode B) inputs, and wiring it into the hwqueue pipeline execution flow.
Changes:
- Replace the “Excel not implemented” placeholder in the HWQueue pipeline with real Excel report generation.
- Add a new
hwqueue_excel.pygenerator that emits formatted Excel workbooks for single-run and sweep data. - Export the new generator functions from
aorta.report.generators.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
src/aorta/report/pipelines/hwqueue_pipeline.py |
Calls the new Excel generator and records the generated file in pipeline results. |
src/aorta/report/generators/hwqueue_excel.py |
New Excel generator for HWQueue data, including basic table conversion and column auto-sizing. |
src/aorta/report/generators/__init__.py |
Re-exports the new HWQueue Excel generator functions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@prosenjitdhole I've opened a new pull request, #128, 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>
|
@prosenjitdhole I've opened a new pull request, #129, 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, #130, 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 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
* Initial plan * Reuse sanitize_table_name/add_excel_table from excel_report.py in hwqueue_excel.py 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>
…g final merge (#128) * Initial plan * Add TODO comments for unused imports/constants to check after final merge 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>
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.
…ide unique table names (#129) * Initial plan * Gate _add_excel_table warnings behind verbose and ensure unique table names 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.