Skip to content

[BENCH-846] Make pooled WER the headline with a mean fallback - #617

Merged
coval-cale merged 2 commits into
mainfrom
cale/bench-846-pooled-headline
Sep 9, 2026
Merged

[BENCH-846] Make pooled WER the headline with a mean fallback#617
coval-cale merged 2 commits into
mainfrom
cale/bench-846-pooled-headline

Conversation

@coval-cale

Copy link
Copy Markdown
Contributor

On the normalized path avg_value for WER is the pooled ratio when every clip in the window carries counts and the per-clip mean otherwise, with the mean always on mean_value; the S/I/D split and series value follow the same rule. Series points expose error_sum and reference_word_sum so consumers can pool across buckets. /v1/leaderboard gains a normalized path behind the reads flag and ranks on the same headline. Methodology doc gains the aggregation section, a dated not-comparable note, and ADR-024.

On the normalized path avg_value for WER is the pooled ratio when every clip
in the window carries counts and the per-clip mean otherwise, with the mean
always on mean_value; the S/I/D split and series value follow the same rule.
Series points expose error_sum and reference_word_sum so consumers can pool
across buckets. /v1/leaderboard gains a normalized path behind the reads flag
and ranks on the same headline. Methodology doc gains the aggregation section,
a dated not-comparable note, and ADR-024.
@coval-cale
coval-cale requested a review from a team as a code owner September 9, 2026 20:24
@coval-cale
coval-cale enabled auto-merge September 9, 2026 20:48

@kdmelon kdmelon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed together with the normalized-table cutover at d90e335. The pooled WER calculation and mixed-history mean fallback check out. All 57 existing aggregate/leaderboard API tests, 22 readiness tests, and changed-file Ruff checks passed. Two targeted PostgreSQL-backed API probes reproduced the issues documented inline; a separate deterministic probe confirmed the inherited #612 validator mismatch. Production cutover readiness was not revalidated.

Comment thread runner/src/coval_bench/api/routers/leaderboard.py
Comment thread runner/src/coval_bench/api/routers/aggregates.py
Comment thread docs/methodology.md
…n 30d

LLM results have no normalized writer, so the reads flag now applies only to
benchmarks the normalized store covers; the leaderboard and all three
aggregates endpoints share the guard. The 30-day compact projection now
carries error_sum and reference_word_sum alongside pooled_value.
@coval-cale
coval-cale added this pull request to the merge queue Sep 9, 2026
Merged via the queue into main with commit 55f7335 Sep 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants