Skip to content

feat: fix Baton adapter retries and publish English results - #16

Open
chenghao-mou wants to merge 3 commits into
mainfrom
chenghao/feat/AGT-3504-baton-benchmark
Open

feat: fix Baton adapter retries and publish English results#16
chenghao-mou wants to merge 3 commits into
mainfrom
chenghao/feat/AGT-3504-baton-benchmark

Conversation

@chenghao-mou

@chenghao-mou chenghao-mou commented Sep 13, 2026

Copy link
Copy Markdown
Member

Follows #13, now merged. Fixes correctness issues in the Baton adapter and publishes the reproduced English results.

Correctness fixes in the adapter: retries classify on a typed HTTP status error instead of digits found in the response body, the capacity and transient budgets are disjoint, empty or malformed events payloads raise instead of scoring every grid point as zero, 16 kHz stereo is down-mixed via the shared audio helpers, and the adapter exposes concurrency like its siblings.

Reproduced results: a fresh run of all 400 English turns gives 12.3%, 4.8%, 577 ms, 350 ms, identical to the README row in #13. Artifacts, comparison report, and site data are committed. The other 13 languages are a follow-up.

Thanks to @Ted-Johnson-JoinIn-AI for the adapter.

Addresses AGT-3504

Initial prompt and agent context

Model: Claude Fable 5.1

Can you review the PR: #13

I have its api key in the .env file, so we can reproduce its claim, and fill in the output artifacts. we only need to fix correctness issues for now, then we can push the results and fixes to the PR. We should do a smoke test first because 300s for a turn detection is unusable for any serious production traffic.

okay, let's create a stacked PR on top of their PR.

🤖 Generated with Claude Code

@chenghao-mou
chenghao-mou changed the base branch from main to feat/baton-turn-detection September 13, 2026 15:39
@chenghao-mou
chenghao-mou changed the base branch from feat/baton-turn-detection to main September 13, 2026 15:40
Retry classification matched digits like 429 anywhere in the error text,
which included up to 400 chars of the response body, so a permanent 422
could be retried for ten minutes. Raise a typed BatonHTTPError and classify
on its status. The capacity and transient budgets are now disjoint.

An empty or malformed events payload now raises instead of scoring every
grid point as p_eot=0.0, matching the other streaming adapters. Audio
decoding goes through the shared helpers so 16 kHz stereo is down-mixed,
and the adapter exposes concurrency (default 8) like its siblings.
Reproduces the README row (12.3%, 4.8%, 577 ms, 350 ms) from a fresh run
of all 400 English turns at concurrency 8 and commits the artifacts under
output/, so the numbers are auditable like every other row. Adds Baton to
the site model config and README adapter docs.
The results table names vendor plus product wherever the product name
alone does not identify the vendor. Adapter ID and run directory are
unchanged.
@chenghao-mou
chenghao-mou force-pushed the chenghao/feat/AGT-3504-baton-benchmark branch from feb94c6 to d9910fe Compare September 13, 2026 15:53
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.

1 participant