Skip to content

fix(bench-intents): bracket more than 26 candidate intents instead of failing - #31

Merged
ghchinoy merged 1 commit into
mainfrom
fix/bench-intents-wide-options
Sep 28, 2026
Merged

ghchinoy merged 1 commit into
mainfrom
fix/bench-intents-wide-options

Conversation

@ghchinoy

Copy link
Copy Markdown
Owner

bench-intents --dataset banking77 sends 30 candidate intents in one choice
question; the server has rejected more than 26 options (single-letter labels)
since before v0.1.0, so every request failed (0/30). With more than 26
options the harness now runs a 2-stage bracket like dgem systemone serve:
balanced groups of <= 20, the top 5 of each group (by probability) go to a
final round; wall time sums both rounds. <= 26 options: unchanged, one call.

Production v0.1.0: banking77 23/30 (keeping top 3 per group: 22; top 8: 23);
clinc150 unchanged. Tests: splitBalanced, bracket round/final calls, summed
wall time, single call at 26 options. Verification run README updated.

Merge after #30 (both touch the top of CHANGELOG.md; I'll rebase if needed).

… failing

bench-intents --dataset banking77 sends 30 candidate intents in one choice
question; the server has rejected more than 26 options (single-letter labels)
since before v0.1.0, so every request failed (0/30). With more than 26
options the harness now runs a 2-stage bracket like dgem systemone serve:
balanced groups of <= 20, the top 5 of each group (by probability) go to a
final round; wall time sums both rounds. <= 26 options: unchanged, one call.

Production v0.1.0: banking77 23/30 (keeping top 3 per group: 22; top 8: 23);
clinc150 unchanged. Tests: splitBalanced, bracket round/final calls, summed
wall time, single call at 26 options. Verification run README updated.
@ghchinoy
ghchinoy force-pushed the fix/bench-intents-wide-options branch from ba01b9e to aadd42a Compare September 28, 2026 23:32
@ghchinoy
ghchinoy merged commit b646a70 into main Sep 28, 2026
3 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.

1 participant