Repository navigation
Correct what actually protects this repo from a cold-init regression - #239
Merged
Merged
Conversation
…ssion
Run 8 said the protection was the pin bump "plus whatever they gate before
tagging". The phonemizer session read it and corrected me: there is no tagging,
they don't cut releases. So it is really whatever they gate before WE choose to
bump, and the protection is the bump being deliberate.
Weaker than the phrasing implied, and the distinction is the point: nothing
upstream fires on a schedule we don't control, so a regression reaches us only
if we pull it.
Also records the shape they offered -- trace-cold takes a goldens directory and
exits non-zero, so it runs against the submodule checkout without depending on
their suite. Verified from the repo root before documenting it:
dotnet run --project external/vernacula-phonemizer/csharp/tools/trace-cold \
-c Release -- external/vernacula-phonemizer/csharp/goldens
traced 189 of 189 languages
no poisons, every traced language carries input spans exit=0
~13s, its own process. Not wired into anything -- recorded because it is the one
gate this repo could own, with a measured cost rather than a guessed one.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KfTCTjC7FTnNm3aS2zszKE
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
1 participant
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.
Docs only. One sentence in Run 8 of the multilingual log was wrong, and the phonemizer session caught it.
I wrote that the protection against a cold-init regression was the pin bump "plus whatever they gate before tagging". There is no tagging — they don't cut releases. So it's really whatever they gate before we choose to bump, and what protects this repo is the bump being deliberate.
That's a weaker guarantee than the phrasing implied, and the distinction is the whole point: nothing upstream fires on a schedule we don't control, so a regression reaches us only if we pull it.
Also records the option they surfaced —
trace-coldtakes a goldens directory and exits non-zero, so it runs against our submodule checkout without depending on their suite. Verified from the repo root before documenting it:~13 s, its own process, 189 languages. Not wired into anything — that's a call for @christopherthompson81, not something to slip into a docs PR. Recorded so the cost is known rather than guessed.
No code change, so the only check that matters is that nothing else referenced the old claim:
grepfinds it in this one line, and theWordSegmentationcomment never made a tagging claim.🤖 Generated with Claude Code
https://claude.ai/code/session_01KfTCTjC7FTnNm3aS2zszKE