Skip to content

feat(swe-bench): all-or-nothing merge gate scoped to exactly one run id - #470

Open
arekay-nv wants to merge 1 commit into
swe-layer-1from
swe-layer-2
Open

feat(swe-bench): all-or-nothing merge gate scoped to exactly one run id#470
arekay-nv wants to merge 1 commit into
swe-layer-1from
swe-layer-2

Conversation

@arekay-nv

Copy link
Copy Markdown
Collaborator

What does this PR do?

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor/cleanup

Related issues

Testing

  • Tests added/updated
  • All tests pass locally
  • Manual testing completed

Checklist

  • Code follows project style
  • Pre-commit hooks pass
  • Documentation updated (if needed)

Stack created with GitHub Stacks CLIGive Feedback 💬

merge_run(wq, run_id) refuses to emit an accuracy number unless every
planned unit has a terminal result, none is abandoned, every unit accounts
for exactly its planned instance IDS (a set comparison, never a count), the
union equals the plan with no cross-shard duplicates, every plan_digest
matches, and no unit carries an infra error. Refusal is a structured
MergeRefusal naming the offending units and ids; there is no force flag and
no partial-credit path.

There is deliberately no --all: merge_run takes a required run id and treats
a foreign run id or digest as a hard error, not a skip.

verify_inventory() cross-checks claims, results and the id-union as
independent producers, so a blind spot shared by one instrument cannot
certify itself.
@arekay-nv
arekay-nv requested a review from a team as a code owner August 26, 2026 03:32
@github-actions

Copy link
Copy Markdown

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@github-actions
github-actions Bot requested a review from nvzhihanj August 26, 2026 03:32
@github-actions github-actions Bot added the size/normal PR Review Policy: <=500 non-test lines & <=20 files label Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/normal PR Review Policy: <=500 non-test lines & <=20 files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants