Eval Review Guide
This guide defines the bounded review contract for public maturity decisions in `aoa-evals`.
Eval Review Guide
This guide defines the bounded review contract for public maturity decisions in aoa-evals.
Use it when an eval bundle already looks useful and reusable, but the repo still needs one explicit decision about whether the bundle is ready for:
draft -> bounded- promotion to
baseline baseline -> canonical
This guide is review-first. It does not auto-promote eval bundles from frontmatter metadata, generated catalog fields, or validator results.
See also:
Outcomes
For draft -> bounded
Bounded review should end in one of two outcomes only:
approve for bounded promotiondefer for now
Use approve for bounded promotion when the eval already reads as a repeatable bounded proof surface with:
- a clear execution path
- a reviewable verdict surface
- example reporting that keeps the public readout honest
- visible failure-vs-readout distinctions
- nearby-bundle boundaries that stay crisp
Use defer for now when the bundle is promising, but the public readout still collapses failure classes, hides interpretation limits, or lacks enough support material to justify bounded status.
For promotion to baseline
Baseline review should end in one of two outcomes only:
approve for baselinedefer for now
Use approve for baseline when the eval already reads as a stable comparison surface within its bounded scope.
Use defer for now when the eval is useful and reusable but still too unstable, too local-shaped, or too weakly interpreted for baseline comparison use.
Record that public approval trail in portable_review.
Do not introduce a separate evidence kind for baseline approval notes.
When a wave materially deepens pairing, shared infra, longitudinal reading, or canonical candidacy,
carry aoa-eval-integrity-check or an equivalent bounded integrity packet as a sidecar.
Bounded Review Axes
| axis | approve signal | defer signal |
|---|---|---|
| execution path clarity | A reviewer can see how the bundle is meant to run and where the bounded read comes from. | The run shape is still too hand-wavy or too dependent on tacit reviewer reconstruction. |
| verdict and readout discipline | Bundle-level verdicts, per-case notes, and example reports stay legible and bounded. | Public readout still jumps too quickly from thin evidence to strong conclusions. |
| failure-vs-readout separation | Failure signals are named directly and stay distinct from the summary language used to report them. | Summary wording smooths over concrete failure signals or lets polished readouts hide them. |
| nearby-bundle distinctness | The bundle clearly says which nearby question it does and does not answer. | The bundle still blurs into composite, diagnostic, artifact, or boundary neighbors. |
| support evidence | Starter evidence, example report, and support notes make the bounded claim inspectable after publication. | Key bounded-claim support still lives only in implied reviewer intuition or scattered prose. |
For baseline -> canonical
Canonical review should end in one of two outcomes only:
approve for canonical promotiondefer for now
Use approve for canonical promotion when the eval already reads as the natural default proof surface within its bounded scope.
Use defer for now when the eval is strong, but still reads as an optional companion, a context-shaped comparator, or an under-validated default.
Baseline Review Axes
| axis | approve signal | defer signal |
|---|---|---|
| repeatability | Runs are stable enough that comparison meaningfully tells us something bounded and non-trivial. | Run-to-run or environment-to-environment movement is still too large for stable comparison. |
| baseline clarity | The bundle clearly states what it compares against and what kind of claim that comparison can support. | The comparison target is muddy, shifting, or easy to over-interpret. |
| fixture discipline | The fixture surface is clearly bounded and representative enough for the stated claim. | Fixtures still look too ad hoc, too narrow, or too entangled with one local context. |
| verdict semantics | Scores or verdicts are explained in a way that a reviewer can understand and not over-read. | The bundle still relies on opaque numbers, vague labels, or hidden judgment. |
| blind-spot disclosure | The bundle honestly names what it does not prove. | The bundle still reads as stronger or broader than its actual surface supports. |
| practical reuse | The bundle can be run outside its birth context with bounded adaptation. | Real use still depends too much on one project's private environment, tooling, or habits. |
Canonical Review Axes
| axis | approve signal | defer signal |
|---|---|---|
| default-use rationale | The bundle clearly reads as the default proof surface for its bounded claim class. | It still reads like one useful option among peers without a crisp default-use rationale. |
| comparison trustworthiness | The bundle has demonstrated stable enough meaning across review cycles, comparisons, or contexts. | Meaning still shifts too much across contexts or review passes. |
| interpretation honesty | Scores, verdicts, and summaries already resist false certainty and inflated claims. | The bundle still invites benchmark theater, false ranking, or over-broad conclusions. |
| portability beyond origin | The eval has shown useful life beyond its birth context or initial local need. | Evidence still comes mostly from one origin, or portability remains too thin. |
| stronger validation than baseline floor | The bundle has more than mere structural completeness; reviews, integrity checks, or repeated use reinforce its strength. | Evidence still looks too close to the minimum baseline floor. |
| fresh public-safety recheck | The current public bundle remains safe, bounded, and suitable for broader recommendation. | Public-safety, local leakage, or interpretation risks remain unresolved. |
Metadata Is Informative, Not Decisive
Stage 1 metadata can support review, but it does not decide it.
maturity_scorecan hint that a bundle is approaching baseline or canonical strength.repeatabilitycan show whether the result surface is stable enough for disciplined comparison.portability_levelcan show whether the bundle still carries strong local shape.validation_strengthcan show whether reinforcement moved beyond structural completeness.blind_spot_disclosurecan show whether the bundle names its own limits.export_readyonly concerns Stage 1 structured catalog publication safety.
None of these fields auto-promote an eval bundle. Baseline and canonical decisions still require explicit human review.
Tracked Review Evidence
Promotion-shaped reviews should leave a small public trail:
- bundles operating at
boundedposture should carry asupport_notethat records the bounded review outcome, failure-versus-readout distinction, and interpretation boundary - bundles already operating at
portable,baseline, orcanonicalposture should carryportable_review; for baseline promotion this is the public review trail - bundles seeking or holding
canonicalposture should carrycanonical_readiness - bundles using
comparative-summaryshould carry asupport_notethat names the comparison contract and its interpretation limits - bundles claiming reusable proof artifacts should ship schema-backed report examples and runner/fixture contracts before those artifacts are treated as part of the public proof surface
These notes do not decide the review outcome by themselves. They keep the public reasoning inspectable after the decision.
Review Notes
Review should stay bounded and concrete.
- Review the bundle as published today, not an imagined future rewrite.
- Name one recommendation only: approve now or defer for now.
- If deferred, name the smallest concrete remaining gap rather than a broad wish list.
- If approved, make the bundle's comparison or default-use rationale explicit enough that later reviewers do not have to infer it from scattered notes.
- Name the exact boundary of the approved claim.
- If the bundle uses scores, say how far those scores can be interpreted before they become theater.
- If the bundle uses categorical verdicts, say what uncertainty remains after the verdict.
Strong reasons to defer
Common reasons to defer include:
- fixture surface still too local or too thin
- score semantics still too vague
- verdict logic still too dependent on hidden reviewer intuition
- blind spots not named clearly enough
- bundle is useful, but still better as a companion than as a default proof surface
- repeated use still reveals unstable meaning
- report summaries imply stronger claims than the bundle truly supports
Recording review outcomes
A bundle-level review note should remain the authoritative place to record:
- outcome
- boundary of the approved or deferred claim
- smallest remaining gap if deferred
- default-use rationale if approved
- any caution about score or verdict interpretation
Related Documents
Annotating documents
lastupdated: "2020-08-04"
Dataset Collection Guide for Yellow Boy Detection
This guide covers collecting and labeling images for training a YOLOv8 model to detect "Yellow Boy" (iron hydroxide precipitate) in water bodies.
redcapAPI (development version)
* `unlockREDCap` upstream dependency on shelter now default to 'SHELTER_' prefix when looking for ENV variable overrides.
Annotating documents
lastupdated: "2020-08-04"