Repository navigation
Add a forward-time historical statement-only backtest - #20
Merged
Merged
Conversation
CmsChase
marked this pull request as ready for review
July 12, 2026 09:06
This was referenced Jul 12, 2026
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.
Summary
This PR completes a focused forward-time historical statement-only backtest after the corrected evaluation controls merged in #17.
It adds:
Frozen comparison
Complete contest start-time buckets are split chronologically at 70/10/20.
index_rank + index_number.Locked result
The frozen contrast is
primary MAE - comparator MAE = -75.9447, about a 15.9% reduction relative only to the predeclared index-only comparator. The paired contest-cluster bootstrap 95% interval is[-86.7339, -65.0783].The test split contains 2,468 problems from 355 contests. Statement parsing coverage is 10,906 / 10,979 overall and 2,422 / 2,468 on test.
Evidence chronology
acea3befroze the backtest design and exact allowlists.f2b979fsealed the 10,979-entry cache manifest.76c69f1added the two-stage runner and leakage guards.1260f4bd54db42ac985797bd99a1d462cc0289e5committed and pushed the validation selection before the locked test output was produced.12b85070180d9170981e8ff4c54b2c5c30980947committed the locked test outputs.18e51dcb106ee70d0ceb7acbddb7f8e54469a504added the result report, coverage artifact, and final-result hash manifest.Git history therefore records one committed locked evaluation after the public selection commit. The output guard prevents overwriting the same output path, but this is not a technical proof that no unrecorded execution occurred elsewhere.
Evidence boundary
This is a forward-time retrospective evaluation, not a prospective test or independent preregistration.
See
docs/HISTORICAL_STATEMENT_BACKTEST_RESULTS.mdfor the full interpretation and artifact links.Validation
git diff --checkpasses.18e51dcb106ee70d0ceb7acbddb7f8e54469a504.TF-IDF is outside this primary result. #18 and #19 are not dependencies and can be closed as superseded after this PR merges.