Repository navigation
check-all.py: 入れ子の check-mutation-claims.py に親の半分の --jobs を渡す(#460) - #467
Merged
Merged
Conversation
…り合わない(#460) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
CR5 で check-all.py から消したのと同じ根拠の無い因果(並列が重なったときに届いた)が、 このコメントに残っていた(/code-review 2 パス目の R5)。計画に 2 パス目の採否の表と、広げた範囲・基準 6 を書く。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Owner
Author
レビュー結果(dev-loop 手順 6)関門
関門は 2 パスで打ち切った。採否の表(1 件 1 行)は 検証して確かめたこと
基準 4(wall が大きく悪化しない)は証拠が弱い同じツリーで交互に測ったのは 1 組だけ(base 248.8 秒 / HEAD 243.1 秒)。交互でない測定のばらつき(217〜262 秒)と、この変更で予想される +8%(約 19 秒)は、この 1 組では判別できない。1 分平均の負荷も 11〜47 と揺れており、統制したのは他の 未証明(この環境では証明できない)
残課題(直していない指摘)
満たしていない受入基準なし(Verifier の反証は全パスで 0 件)。ただし基準 4 は上のとおり証拠が弱い。 |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Fixes #460
何を変えたか
scripts/check-all.py—check-mutation-claims.pyを起こすときだけ--jobs max(1, 親の --jobs // 2)を渡す。親が--jobs 1なら入れ子も直列。docstring に、半分にした根拠となる実測(10 コアの macOS 1 台)と、打ち切りの原因が特定できていないことを書いたscripts/test-check-all.py— 契約 9: 入れ子に渡す--jobsが親より多くなく、親 1 なら 1 であることを、値ではなく性質で見る。変異 3 件scripts/test-worktree-scripts.py— コメントの 1 文だけ。根拠の無い因果を、記録にある事実だけに直した(2 パス目の R5)docs/plans/issue-460.md— 計画・実測・採否の表プラグインの中身は変えていないので版は上げていない。反映経路なし(
scripts/だけ)。検証
python3 scripts/check-all.py: 51/51(失敗 0)。最後に回したのはbba67c9の時点dev-loop:dev-loop-verifier)と/code-review(high)を 2 パス。それに加えて、2 パス目の修正に狭いパスを 1 回。結果と採否の表は PR コメントに書く🤖 Generated with Claude Code