Skip to content

data: export review-eval ground truth (unblocks the model A/B — qaEngineer#20)#21

Merged
mabry1985 merged 1 commit into
mainfrom
data/export-review-eval-truth
Jul 24, 2026
Merged

data: export review-eval ground truth (unblocks the model A/B — qaEngineer#20)#21
mabry1985 merged 1 commit into
mainfrom
data/export-review-eval-truth

Conversation

@mabry1985

Copy link
Copy Markdown
Member

Blocker 2 of the qaEngineer critical path. The 24 hand-grounded findings were gitignored at data/review-eval/, local to the qaEngineer box — the lab side had only a 3-row truth.seed.jsonl and can't score the model A/B without the rest. This commits the real file so the lab can git pull it.

What's in it

  • truth.seed.jsonl — 24 labeled rows, heads resolved to full 40-char SHAs so a replay run's findings join cleanly on (repo, head, file, line). The original short SHAs (7–8 char, inconsistent) would have been a join hazard.
  • README.md — schema + the scoring notes that actually matter:
    • precision must exclude assertion_only / not_grounded rows (findings I didn't independently ground)
    • recall must include the false_negative rows — the defects the panel missed (protoAgent#2143's orphan, #2210's event-loop major). A replay of the same head should re-find these; a model that doesn't scores a miss.
    • honesty = disregarded_evidence count + fabricated-quote survival rate
  • build_ds.py + the original file, for provenance.

Seed: 18 true · 3 false · 1 false-negative · 2 unverified

Secret-scanned — labels and claims about our own PRs, nothing sensitive. The gitignore said "local until it's big enough to move"; the A/B needs it now.

Replay mode (blocker 1, the plugin-side runner) is the next PR.

🤖 Generated with Claude Code

…qaEngineer#20)

The 22+ hand-grounded findings were gitignored at data/review-eval/, local to
this box — the lab side had only a 3-row truth.seed.jsonl and couldn't score the
model A/B without the rest. Committing the real file so the lab can pull it.

- `truth.seed.jsonl` — 24 labeled rows with heads resolved to full 40-char SHAs
  (the short SHAs would not join cleanly against a replay run's full-SHA output).
- `README.md` — the schema + the scoring notes that matter: precision must
  exclude `assertion_only`/`not_grounded` rows, recall must include the
  `false_negative` rows (the defects the panel missed — #2143, #2210), honesty =
  disregarded-evidence + fabricated-quote rate.
- `build_ds.py` + the original short-SHA file, for provenance.

Secret-scanned: labels + claims about our own PRs, nothing sensitive. The
gitignore comment said "local until it's big enough to move" — the A/B needs it
now, so it moves.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mabry1985
mabry1985 merged commit a3bda77 into main Jul 24, 2026
1 check passed
@mabry1985
mabry1985 deleted the data/export-review-eval-truth branch July 24, 2026 10:11
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