[WIP] Integrate multi-judge metrics into TUI - #523
Conversation
| bias_gap_str = ( | ||
| f" Bias gap: [bold]{float(bias_gap):.3f}[/bold]" | ||
| ) | ||
| except (TypeError, ValueError): |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
@claude[agent] fix the CI Checks |
Two CI jobs were failing. 1. Linting and Formatting — 2. Commit Messages — Preview of the TUI change (the |
|
@claude[agent] fix the CI Commit Checks |
I attempted this but I'm blocked by tooling — the commit check can't be fixed from my side. What I tried: squashed all 5 commits into a single compliant commit ( The 4 offending commits lack a cz_gitmoji type prefix — note
Two ways to resolve: A. Squash-merge (no local work) — merge with a compliant title, e.g. B. Force-push the squash: git checkout claude/integrate-multi-judge-metrics
git reset --soft 8118622
git commit -m "✨ feat(tui): show multi-judge metrics in run summary"
git push --force-with-leaseEither way the code itself is unchanged and green: |
|
@claude[agent] fix the CI Commit Checks |
This one needs you — I can't fix it, and retrying won't change that. Latest run 30203967056 on Why I'm blocked: Fastest fix — squash-merge the PR with title Or force-push: git fetch origin claude/integrate-multi-judge-metrics
git checkout claude/integrate-multi-judge-metrics
git reset --soft 8118622
git commit -m "✨ feat(tui): show multi-judge metrics in run summary"
git push --force-with-leaseNote |
Agent-Logs-Url: https://github.com/AISecurityLab/hackagent/sessions/25caf2e3-9c98-45d1-95a2-77300db3bc42 Co-authored-by: franconicola <51865029+franconicola@users.noreply.github.com>
Agent-Logs-Url: https://github.com/AISecurityLab/hackagent/sessions/25caf2e3-9c98-45d1-95a2-77300db3bc42 Co-authored-by: franconicola <51865029+franconicola@users.noreply.github.com>
Agent-Logs-Url: https://github.com/AISecurityLab/hackagent/sessions/25caf2e3-9c98-45d1-95a2-77300db3bc42 Co-authored-by: franconicola <51865029+franconicola@users.noreply.github.com>
Agent-Logs-Url: https://github.com/AISecurityLab/hackagent/sessions/63505e61-b0d7-4dbd-bcfe-2066953a0def Co-authored-by: franconicola <51865029+franconicola@users.noreply.github.com>
ad560fe to
430b6e6
Compare
There was a problem hiding this comment.
Pull request overview
This PR extends the Results TUI run summary header to surface multi-judge evaluation metrics so they’re visible in the terminal UI (aligning with what’s already available in the dashboard), addressing #279.
Changes:
- Render per-judge strictness values and bias gap in the Results TUI run summary when
is_multi_judgeis true. - Add integration tests to verify multi-judge metrics are displayed (and that strictness/bias gap are hidden for single-judge runs).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
hackagent/cli/tui/views/results.py |
Adds conditional rendering of per-judge strictness and bias gap to the run summary header panel. |
tests/integration/tui/results/test_widget_lifecycle.py |
Adds integration coverage for the run summary header to validate multi-judge metrics rendering behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.