Skip to content

Record review timing and correction outcomes in benchmark results - #226

Open
morgaesis wants to merge 1 commit into
mainfrom
fix/adjudication-release-evidence
Open

morgaesis wants to merge 1 commit into
mainfrom
fix/adjudication-release-evidence

Conversation

@morgaesis

Copy link
Copy Markdown
Contributor

Benchmark results omit review timing and correction outcomes, leaving deadline use unverified. Retain bounded, nonce-bound terminal measurements for each original attempt, distinguish explicit zero-candidate results (notRequired) from missing failed-attempt evidence (unknown), and require complete final successful capture before calibration recording or release comparison.

Keep diagnostic transport separate from retry classification: preserve unmatched diagnostic bytes and retain a newline boundary when removing matching capture. Offline tests cover short writes, malformed and replayed frames, retry decisions and accounting, and actual CLI children against a local mock endpoint. Genuine calibration and release qualification require fresh runs against the changed evaluator.

Retain nonce-bound terminal measurements for each original benchmark attempt,
including explicit zero-candidate results and unknown failed-attempt capture.
Validate final successful capture before calibration recording or release
comparison, with diagnostic transport isolated from existing retry decisions.

@postil-dev postil-dev Bot 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.

error 1 blocking finding open
1 finding posted inline

Review details

Comment thread bench/src/compare-baseline.ts
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