Repository navigation
[MLPerf 6.1][GRPO] Define Qwen3.5 OpenHands agent harness contract - #599
Merged
ShriyaRishab merged 7 commits intoSep 30, 2026
Merged
ShriyaRishab merged 7 commits into
ShriyaRishab merged 7 commits into
Conversation
Signed-off-by: Al Rigazzi <arigazzi@nvidia.com>
Signed-off-by: Al Rigazzi <arigazzi@nvidia.com>
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
RissyRan
approved these changes
Sep 29, 2026
Resolve conflict in the benchmark-specific rules: both master (offline evaluation rule) and this branch (OpenHands agent harness contract) add a qwen35_397b_grpo entry at the same location. Keep both texts under a single qwen35_397b_grpo heading. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
jepio
approved these changes
Sep 29, 2026
ShriyaRishab
previously approved these changes
Sep 30, 2026
ShriyaRishab
approved these changes
Sep 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CodeActAgentharness contract forqwen35_397b_grpo.security_riskoptional forexecute_bashandstr_replace_editor, and require disclosure of the harness repository, commit, and all reference deviations.Motivation
The current rule pins the agent harness to a single OpenHands repository and commit. Integrating the harness with different training, inference, serving, or environment frameworks may require narrow implementation changes for compatibility, reliability and failure handling, resource isolation, or security. An exact commit requirement makes behaviorally equivalent implementations noncompliant even when they preserve benchmark-relevant behavior. This change instead fixes the model-visible inputs, tool-call contract, action and result semantics, turn accounting, termination behavior, and scoring contract, while allowing a submitter-selected immutable implementation and requiring every deviation from the reference to be disclosed.
We propose making
security_riskoptional because it is auxiliary, model-generated metadata that is not required for tool execution or task scoring in this benchmark's noninteractive configuration. The reference OpenHands tool schemas declaresecurity_riskrequired forexecute_bashandstr_replace_editor, but the reference runtime does not enforce its presence: it checkscommandforexecute_bash, checkscommandandpathforstr_replace_editor, and setssecurity_riskonly when supplied. In the benchmark's noninteractive configuration,confirmation_modedefaults tofalse, risk handling is conditional on confirmation mode, and task reward depends only on whether the task is resolved. Its surrounding syntax accounts for a substantial fraction of observed rollout–trainer logprob outliers, which can cause entire successful trajectories to be discarded. Making the annotation optional removes the requirement to generate those problematic contexts while preserving task semantics, tool capabilities, execution controls, and training filters. Preliminary multi-seed results support the mitigation.Validation
git diff --check origin/master...HEADpasses.