Uh oh!
There was an error while loading. Please reload this page.
docs(eval): report the three-arm edit-contract comparison - #3158
Draft
Astro-Han wants to merge 1 commit into
Draft
docs(eval): report the three-arm edit-contract comparison#3158Astro-Han wants to merge 1 commit into
Astro-Han wants to merge 1 commit into
Conversation
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
One harness, one model, one composition, three file-editing plugins. The arms score 56, 56 and 53 of 86 tasks, and the run has no power to separate them: with 17 to 22 discordant pairs per comparison its minimum detectable effect is about 13 tasks, and against the differences actually observed — zero and three — its power is 3% to 7%. The report says that rather than reporting the three non-significant p-values as a finding of equivalence. The 95% intervals are the honest summary: each spans roughly 22 percentage points, so every pair is as consistent with no difference as with one arm being nine or ten tasks better. The arms disagree on 29 of 86 tasks. No arm was run against itself, so this data cannot say how much of that is variance and how much is a real per-task effect; a same-arm repetition is named as the cheapest experiment that would make the rest of it interpretable. Records what the arms differ in, none of it powered: fs nearly doubles the baseline's verification failures while losing fewer cells to the deadline, spends 30% more reasoning, and costs 20% more per pass. Records that the treatment is a tool family rather than a diff format — fs presents four tools and the least instruction text of the three. Four harness failures found and fixed during the run are written up as operational findings, because each is a prerequisite for reproducing it: the toolchain's base image is a glibc floor for the task images, a task instruction beginning with a dash needs two argument separators, inotify instances are a host-wide budget that caps concurrency, and verifier bandwidth is a scheduling constraint for tasks that build an environment. Three tasks are unscored. The two torch tasks are reported as unscored rather than as zeros: an agent that exhausted its budget may still have left a passing state behind, and the verifier that would have said so never ran. Generated-by: Claude Code
M4n5terforce-pushed
the
docs/eval-edit-contract-arms-report
branch
from
August 26, 2026 10:03
20f4778 to
b8a83deCompare
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 freeto 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
Adds the report and per-task CSV for the three-arm edit-contract experiment on Terminal-Bench 2.1 with deepseek-v4-flash: one harness, one model, one composition, and the file-editing tool contract as the only variable (
str_replace_editorbaseline vs@deepseek-ai/dsh-tool-fsvs a repo-authored V4Aapply_patchplugin).Headline results, 86 scored tasks per arm:
str_replace_editor56,apply_patch56,fs53 — and the run has no power to separate them. Exact McNemar p = 1.00 / 0.63 / 0.65 with 95% CIs of roughly ±11 pp; the minimum detectable effect is ~13 tasks and power against the observed spread is 7%. The report states this as a failure to detect, not a finding of equivalence.--parsing,fs.inotify.max_user_instancesas a host-wide budget, verifier bandwidth as a scheduling constraint) with their fixes.Verification
Docs-only. Numbers are derived from the archived run results (5 run directories, later runs superseding earlier cells); the analysis script cross-checks totals against the per-arm token columns. CSV records per-task per-arm outcome, status, seconds, cost, requests and source run.
AI use
Select exactly one:
Tool(s) and scope: Claude Code ran the experiment harness, performed the statistical analysis, and authored the report, CSV and this PR body. The commit carries a Generated-by trailer. A human contributor reviews the final text and owns the merge decision.
Checklist
Does this PR entail a change in behavior?