Repository navigation
feat(report): list every regressed example in Top regressions, one card each - #65
Merged
Merged
Conversation
…rd each The list showed the five most negative scored rows, one per example and evaluator, so one example scored -1 by four evaluators filled four of five slots and drops inside an evaluator's own threshold (scored as no change by the analysis) were listed as regressions. Cards are now per example, worst first, capped at 50 per prompt; within-threshold drops sit in a closed line below. The report also gets more room between sections and blocks. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…aiming regressions Review of the previous commit: a card in the within-threshold list still read "Why flagged" in red and said the meaning drifted, and the per-example breakdown tinted every negative row, so a run with 15 regressed examples showed 21 red rows. Within cards now read "Within threshold" in a neutral tone; breakdown rows carry regressed, are tinted only when an evaluator dropped beyond its threshold, and quote that drop; forgiven drops show muted. Adds tests for tie order, the within list's cap and an empty regression list; fixes two stale comments and the report.json doc wording. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 22, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this changes
Top regressions lists every example that regressed, one card per example.
MAX_REGRESSION_CARDS = 50per prompt;A drop inside an evaluator's own threshold is not listed as a regression.
analyzealready tests asemanticrow abovemin_similarityas no change (analysis.tolerance.within_tolerance). Yet the list showed a 0.93 similarity, with a 0.9 threshold, as a regression.The per-example breakdown uses the same rule.
More spacing.
report.json.top_regressionskeeps its per-row shape: one row per example × evaluator, for the examples shown.regressed_example_count,within_threshold_regressions,within_threshold_example_count, andexample_rows[].regressed.report.json, and the hosted server builds its own regression list, so nothing hosted changes.Results on real runs
Review
An independent review approved this with fixes, all now in the second commit:
report.jsondoc wording was imprecise.Known trade-off: the report grows with the number of regressed examples. On the planner run,
report.htmlwent from 115 KB to 261 KB.Not changed here: the narrative's regression samples (
insights/stage.py) still pick examples by raw worst delta. So the narrative can still describe a within-threshold drop as a regression. That's a separate follow-up.Test plan
make ci: ruff, ruff format,mypy --strict, and pytest (3393 passed, 95.3% coverage)pre-commit run --all-filesTestTopRegressionsListEveryRealRegression, each written first and seen to fail:report.jsonfields.🤖 Generated with Claude Code