fix(perf)!: unify local and release performance workflows - #210
Conversation
- Retain local Markdown, CSV, and provenance artifacts while allowing same-version comparisons of tracked changes. - Exclude untracked files explicitly and isolate the narrowed non-exact comparison bundle. - Promote distinct-release artifacts through performance-doc and keep performance-release atomic and rollback-capable. - Render one consolidated table per benchmark suite and reject invalid release publication states. BREAKING CHANGE: rename performance-rerender to performance-doc and performance-local-vs-linalg to performance-local-non-exact, replace --rerender with --promote-artifacts, and remove performance-archive-published.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (14)
📝 WalkthroughWalkthroughThe PR updates performance workflows to retain validated CSV and provenance artifacts, support separate local and documentation-only commands, validate release identifiers, promote reports without rerunning benchmarks, and render consolidated suite-level comparison tables. ChangesPerformance artifact workflow
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant CLI
participant archive_performance
participant performance_artifacts
participant bench_compare
participant ReleaseArchive
CLI->>archive_performance: Generate comparison
archive_performance->>performance_artifacts: Export CSV and provenance JSON
archive_performance->>performance_artifacts: Reload and validate artifacts
performance_artifacts->>bench_compare: Provide comparison inputs
bench_compare-->>archive_performance: Render Markdown report
CLI->>archive_performance: Promote retained artifacts
archive_performance->>ReleaseArchive: Archive promoted report
Possibly related issues
Possibly related PRs
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #210 +/- ##
=======================================
Coverage 97.88% 97.88%
=======================================
Files 8 8
Lines 5007 5007
=======================================
Hits 4901 4901
Misses 106 106
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
BREAKING CHANGE: rename performance-rerender to performance-doc and performance-local-vs-linalg to performance-local-non-exact, replace --rerender with --promote-artifacts, and remove performance-archive-published.
Summary by CodeRabbit
New Features
performance-docworkflow for generating and promoting documentation-ready reports.Documentation
Bug Fixes