Skip to content

feat(metric-benchmark): specify and validate experiment results - #105

Merged
mstaats-nvidia merged 23 commits into
mainfrom
feat/metric-benchmark-results
Oct 28, 2026
Merged

mstaats-nvidia merged 23 commits into
mainfrom
feat/metric-benchmark-results

Conversation

@mstaats-nvidia

@mstaats-nvidia mstaats-nvidia commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Specify the results that benchmark execution must produce, and add validation code that checks those results against both the upstream benchmark definition and the results-format constraints.

results-format.md defines the directory layout, experiment and case JSON indexes, completeness and attempt rules, and native Harbor evidence requirements. A mock results directory illustrates one baseline and two mutants across three tasks. validation-report.md describes the validator’s output and includes the corresponding example report.

The validator checks that configured cases, agents and tasks are accounted for and that recorded outcomes agree with the prepared inputs and native evidence. A thin CLI calls the Python library, validating all cases by default or one case selected with --case, using the same experiment output root. It writes a report only after all selected cases pass; missing observations require explicit reasons rather than being treated as zero scores.

Tests cover the example, CLI/library agreement, invalid evidence, relocation, and case selection. Validation completed on the final revision: full local suite 2,596 passed and 25 skipped; the focused configuration/signing/validation run passed all 82 tests. Ruff lint/format, type checks for Python 3.12 and 3.13, copyright, dependency licenses, whitespace, and commit sign-off checks passed. Hosted CI passed, including both Python test jobs. Local skips cover unavailable Node rendering, optional external/live runtimes, and Windows-specific behavior.

This follows the merged benchmark-definition PR #94. The execution runner and analysis stages remain in downstream draft PRs #106, #95, #96, #97, and #99; their callers need to follow the revised results schema when the stack is updated.

@mstaats-nvidia
mstaats-nvidia added this pull request to stack #107 October 21, 2026 21:10
@mstaats-nvidia
mstaats-nvidia force-pushed the feat/metric-benchmark-results branch from b398fe2 to 62418ad Compare October 23, 2026 14:33
mstaats-nvidia added a commit that referenced this pull request Oct 23, 2026
Defines an experimental benchmark for evaluating how well our coverage
metric predicts evaluation effectiveness through mutation testing.

The concise [root
README](https://github.com/NVIDIA-NeMo/eval-author/blob/docs/metric-benchmark-design/labs/metric-benchmark/README.md)
describes the project, its work-in-progress status, and the expectation
that case examples will grow, change, or be dropped as we build in plain
sight. Prep is external to this repository; runs consume prepared case
examples, fixed subset manifests, and a YAML benchmark definition.

The [stage
descriptions](https://github.com/NVIDIA-NeMo/eval-author/blob/docs/metric-benchmark-design/labs/metric-benchmark/docs/stages.md)
focus on four executable stages: run mutants × eval tasks, compute
mutant scores, compute coverage, and correlate the score pairs. The
first execution script will be a basic local Harbor loop; NeMo Helix
support is planned afterward. The [architecture and both SVG
figures](https://github.com/NVIDIA-NeMo/eval-author/blob/docs/metric-benchmark-design/labs/metric-benchmark/docs/architecture.md)
explain the inputs, component boundaries, shared results directory, and
statistical comparison.

This is the documentation-only base of the stack. Implementation PRs #94
→ #105 → #106 → #95 → #96 → #97 → #99 inherit the overview and design
unchanged. Their detailed CLI guide lives separately in `docs/tools.md`.

Validation: the five files in this PR are Markdown/SVG only. Local links
and anchors, Markdown fences, headers, SVG XML, and whitespace checks
pass; the updated figure was rendered and visually inspected. Restacking
preserves all source and test files byte-for-byte. No live evaluations
or model calls were made.
@mstaats-nvidia
mstaats-nvidia force-pushed the feat/metric-benchmark-results branch 2 times, most recently from 5c4b80b to f1792e4 Compare October 26, 2026 12:32
@mstaats-nvidia
mstaats-nvidia force-pushed the feat/metric-benchmark-results branch from f1792e4 to f2b8e4d Compare October 26, 2026 13:43
@mstaats-nvidia
mstaats-nvidia force-pushed the feat/metric-benchmark-results branch 11 times, most recently from 1e88bb2 to 9b95325 Compare October 27, 2026 16:27
Signed-off-by: mstaats <mstaats@nvidia.com>
Add a checked-in nine-trial results directory matching the prepared input example. Validate its expected report through the library and CLI, and document a reading order for the index and native evidence.

Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
…inputs

Signed-off-by: mstaats <mstaats@nvidia.com>
…vidence

Signed-off-by: mstaats <mstaats@nvidia.com>
@mstaats-nvidia
mstaats-nvidia force-pushed the feat/metric-benchmark-results branch from c536441 to f2a12ab Compare October 27, 2026 21:04
…rray

Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
…rays

Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
Signed-off-by: mstaats <mstaats@nvidia.com>
@mstaats-nvidia mstaats-nvidia changed the title feat(metric-benchmark): validate the results directory contract feat(metric-benchmark): specify and validate experiment results Oct 28, 2026
@mstaats-nvidia
mstaats-nvidia marked this pull request as ready for review October 28, 2026 12:40
@mstaats-nvidia
mstaats-nvidia requested a review from a team as a code owner October 28, 2026 12:40
Signed-off-by: mstaats <mstaats@nvidia.com>
@mstaats-nvidia
mstaats-nvidia merged commit 219888c into main Oct 28, 2026
19 checks passed
@mstaats-nvidia
mstaats-nvidia deleted the feat/metric-benchmark-results branch October 28, 2026 13:01
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.

2 participants