| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
This PR brings the repository’s published LiveMathematicianBench split metadata back in sync with the already-corrected checked-in item files (35 train / 17 val / 125 test), and adds a regression test to prevent future manifest/item count drift.
Changes:
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tests/test_data_manifests.py | Adds a consistency test ensuring manifest counts match checked-in split item files. |
| data/README.md | Updates released-splits documentation for LiveMathematicianBench to 35 / 17 / 125. |
| data/livemathematicianbench_id_split/split_manifest.json | Fixes LiveMathematicianBench declared split counts to align with checked-in items. |
| .osc-metadata/sync.json | Adds sync metadata state file (appears unrelated to split-metadata fix). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
| { | ||
| "fork_synced_at": "2026-07-26T09:38:34.869801+00:00", | ||
| "commits_behind_before_sync": 210, | ||
| "action_taken": "synced" | ||
| } No newline at end of file |
|
Appreciate you merging the LiveMath split metadata sync, and thanks for spotting which item actually belonged in the test split. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
The maintainer identified one LiveMathematicianBench item that belonged in the test split and moved 202512:46 on current main, so the checked-in item files now contain 35 train, 17 validation, and 125 test records. The accompanying split_manifest.json and released-splits table still advertise 35/18/124, leaving the repository's canonical metadata inconsistent with the corrected paper split. Because the hard scorer averages binary per-example results, using the corrected 125-item test file resolves the reported LiveMath denominator mismatch, but consumers following the stale metadata can still materialize or report the wrong split sizes. There are no competing or closed-unmerged cross-referenced PRs.
Summary
Update the LiveMathematicianBench manifest counts and the corresponding data/README.md released-splits row to 35/17/125, preserving the item move already present on main and avoiding unsupported claims about the other benchmarks' run aggregation. Add a focused tests/test_data_manifests.py consistency test that discovers each checked-in data/*/split_manifest.json, follows the canonical train/val/test layout, and compares every declared count with the length of its items.json array. This uses the manifest as the published contract and the checked-in item files as the implementation source of truth, covering the data-to-documentation wiring without changing the runtime scorer or benchmark loader.
Validation
Fixes #158