Repository navigation
Make lcode bench --compare say when attempts are missing - #253
Merged
Merged
Conversation
A task with one attempt counted as certain, so the noise came out too small, and a run that lost 18 of its 72 attempts compared as usual. A single attempt's pass rate now comes from both runs, and --compare warns when the two runs didn't have the same number of attempts. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts: # CHANGELOG.md
Merged
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.
Found on 0.61.0's gate run. uv's cache was emptied while the run was going, so its last 18 SWE-bench attempts couldn't start.
lcode bench --comparestill gave a normal verdict and exit 0, and its noise came out too small.What was wrong
--comparescores only the attempts that ran, which is right, but it didn't mention that one side had 54 attempts and the other 72.What changes
--comparewarns when the two runs didn't have the same number of attempts over the tasks in both. The exit code still depends only on the verdict.Two single attempts that disagree used to be "a drop beyond the noise (±0)". Now they're within the noise.
Real test
0.60.0's run against the night's 0.61.0 run, the same two folders:
Not the same number of attempts ran: 72 before and 54 after, over the tasks in both. With fewer attempts, a result says less.New tests cover both cases. The full suite,
ruff,basedpyrightandmkdocs build --strictpass. This doesn't change anything the model sees.🤖 Generated with Claude Code