AORTA-23 : Phase 2 : Implement blank pipeline command and CLI updates#99
AORTA-23 : Phase 2 : Implement blank pipeline command and CLI updates#99prosenjitdhole wants to merge 9 commits intoprosenj_cli_hq_eval_report_phase_1from
Conversation
There was a problem hiding this comment.
Pull request overview
Adds initial scaffolding for an hwqueue report pipeline, wiring it into the existing aorta-report pipeline CLI so hw_queue_eval JSON inputs can be loaded and a (future) report workflow can be orchestrated.
Changes:
- Introduces
HWQueuePipelineConfig/HWQueuePipelineResultandrun_hwqueue_pipeline()with single-input and baseline-vs-test comparison entry points. - Adds
aorta-report pipeline hwqueueCLI subcommand with options for single file or directory comparison modes. - Exposes the new HWQueue pipeline symbols via
pipelines/__init__.py.
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 |
New HW queue pipeline module with config/result dataclasses and placeholder step orchestration. |
src/aorta/report/pipelines/cli.py |
Adds pipeline hwqueue subcommand and user-facing CLI output for the pipeline run. |
src/aorta/report/pipelines/__init__.py |
Re-exports HWQueue pipeline entry points/types from the pipelines package. |
💡 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, #126, 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, #127, 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 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@prosenjitdhole I've opened a new pull request, #135, to work on those changes. Once the pull request is ready, I'll request review from you. |
* Initial plan * Fix HWQueue import style in cli.py to be consistent with other pipeline commands 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>
…ne (#126) * Initial plan * Add optimization TODO for duplicate find_common_workloads() call 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.
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, #136, 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 no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hw_q pipeline implementation