Skip to content

fix: principled abstention threshold (closes #33) — validated on three question sets - #35

Merged
vedantggwp merged 1 commit into
mainfrom
fix/abstention-threshold
Jul 10, 2026
Merged

fix: principled abstention threshold (closes #33) — validated on three question sets#35
vedantggwp merged 1 commit into
mainfrom
fix/abstention-threshold

Conversation

@vedantggwp

Copy link
Copy Markdown
Owner

The held-out eval (#34) showed abstention was overfit (0/2 on fresh absent topics). This replaces it with a corpus-relative threshold (score statistics + salient-term coverage, every constant justified in comments), validated with an anti-overfit protocol:

Set Covered top-1 Absent abstain
Tuned (PR #21) 8/8 2/2
Held-out (#34) 9/10 2/2 (was 0/2)
Third set, pre-registered before implementation n/a 5/6 (bar ≥5/6)

The 'abstains instead of hallucinating' property now holds on questions the threshold has never seen. Implemented by codex (gpt-5.5); independently re-run and verified here.

…three question sets

Closes #33. No-hit decision now derives from corpus-relative score
statistics + salient-term coverage instead of constants implicitly
calibrated to the tuned set's absent questions. Validation: tuned set
2/2 abstain (kept), held-out 2/2 (was 0/2), pre-registered third set
5/6 (bar: >=5/6; pre-registration timestamped before implementation).
Covered retrieval preserved: held-out 9/10 top-1, 10/10 top-3.
369 tests; bundles rebuilt. Implemented by codex from spec;
independently re-verified here.
Copilot AI review requested due to automatic review settings July 10, 2026 18:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vedantggwp
vedantggwp merged commit b42807c into main Jul 10, 2026
1 check passed
@vedantggwp
vedantggwp deleted the fix/abstention-threshold branch July 10, 2026 18:01
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