You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Say what the forced insertion is worth, in numbers that were measured
Third pass at this comment, and the first two both overstated the same
half from samples too small to carry the claim. What it said and what
measurement says:
* "the conversion from sampled to fired is 100%" -- it is about four in
five (350 of 450 aligned examples over 24 randomized runs of 250).
The conclusion it supported was right and is kept; the number is
gone, replaced by the decline paths, which are all the stage
deciding: a FAMILY_COMMA opts the stage out, or an earlier
script-written token takes the surname site -- a bare drawn surname
('김 김민준'), a hangul post-nominal, or a Han token under a policy
that activated HAN. That last group is what the rewrite deleted and
this restores; a Latin word never takes the site (zero occurrences in
1082 aligned examples), so the note that only Latin blocks it named
the one blocker that does not bite.
* "13-19 [per] 250" alignment, "0-4 times" in the name, "one entry
among ~30", "0-5 per run" for the peel -- measured 5.5% of draws over
20000, a couple per 250, a pool of median size 23, and 0-7 peels per
randomized run. Ranges taken from a handful of runs are dropped or
widened.
* "roughly eight times what it was" -- follows from no measurement I
can reproduce; dropped for the fire counts themselves.
Leads instead with the one figure that cannot rot: instrument _split
under the committed derandomize=True seed and origin/master counts zero
surname splits against two peels, this tree 23 against one. One
command, same answer every time.
Adds what the change does NOT buy, which is the honest headline: this
layer asserts totality and span-exactness only, so every behavioral
mutant in the stage passes here before and after and is killed by
tests/v2/pipeline/test_script_segment.py. The one defect class it can
catch is span arithmetic in _split, and the peel already reached that
path -- writing `base + end` as `base + end + 1`, origin/master finds
it in 8 runs of 10 at a median 1561 shrink calls and 14.6 seconds,
this tree in 10 of 10 at 346 calls and 3.6 seconds.
And two costs that went unmentioned: the insertion drops the stage's
i-is-None early return under a live configuration from 1.3% of
examples to 0.1% (it survives where a drawn quote pair carries the
token into a nickname), and an insertion at the end can suppress a
peel, since the peel site is the last non-post-nominal token.
Correction to this branch's first commit message, which is pushed and
not being amended: it gave the pre-change baseline as "0-2" where the
comment said 0, 1, 1, 2, 2, 4. Neither is the range -- 36 randomized
runs of 250 give 0-9.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
0 commit comments