Skip to content

Commit e9569e9

Browse files
committed
ci(test-shards): the self-test verdict names the closure allowance when the bins exceed the bare cap
The success line said "within the <cap> density cap" from the bare cap alone; once a dataset slices only through the allowance (bins 1553 s against a 1455.81 s cap on the 2026-11-02 refresh) that sentence would be false. It now names the allowance exactly when the bare cap did not decide, so today's line is byte-identical. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ihL2gLghEqeDKTbDTC7Zd
1 parent 7e9f788 commit e9569e9

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

‎scripts/partition-test-shards.mjs‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3878,8 +3878,12 @@ function selfTest() {
38783878
`partition-test-shards: self-test OK (${datasetPackages.length} measured packages ` +
38793879
`-> ${datasetItems.length} shard items, ${SHARD_COUNT} shards, ` +
38803880
`max/mean ${balance.ratio.toFixed(2)}x <= ${MAX_SHARD_OVER_MEAN}x, floor ${balance.floor.toFixed(0)}s, ` +
3881-
`bins ${balance.totals.map((t) => t.toFixed(2)).join('/')}s within the ${datasetCap.toFixed(2)}s density cap, ` +
3882-
`whole legs ${wholeLeg.sums.map((t) => t.toFixed(0)).join('/')}s within ${WHOLE_LEG_CAP_SECONDS}s, ` +
3881+
`bins ${balance.totals.map((t) => t.toFixed(2)).join('/')}s within the ${datasetCap.toFixed(2)}s density cap` +
3882+
// Named only when the bare cap did not decide, so the line never claims a cap the bins exceed.
3883+
(balance.max > datasetCap
3884+
? ` plus its ${closureAllowance(datasetItems, SHARD_COUNT).toFixed(2)}s declaration-closure allowance`
3885+
: '') +
3886+
`, whole legs ${wholeLeg.sums.map((t) => t.toFixed(0)).join('/')}s within ${WHOLE_LEG_CAP_SECONDS}s, ` +
38833887
`file-level slices: ` +
38843888
`${datasetPlan.decisions.filter((d) => d.count > 1).map((d) => `${d.name} x${d.count}`).join(', ') || 'none'})`
38853889
);

0 commit comments

Comments
 (0)