Repository navigation
fix(asr): Preserve names, possessives and surrounding words - #1041
Open
achembarpu wants to merge 2 commits into
Open
achembarpu wants to merge 2 commits into
achembarpu wants to merge 2 commits into
Conversation
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.
Why is this change needed?
Vocabulary rescoring can duplicate already-correct names, remove possessive endings, or consume the word following a hyphenated full name.
Examples:
mister QuilterbecomesQuilter Quilter.mister Quilter's mannerloses its possessive.Chandra Shankar-Solansky saiddeletessaid.What changes?
Legitimate corrections remain available, including longer full-name corrections that preserve recognized prefixes. Coarse overlapping timestamps do not establish acoustic ownership. The production logic contains no special-case names or verbs.
Add licensed real-speech fixtures and regression coverage for name preservation, possessives, repetitions, aliases and hyphenated name boundaries.
Validation
saidin the two affected v2 outputs.The broader streaming-rescue test class still reproduces the existing upstream
Hey → Codexfalse positive and was excluded from the passing selection. Other acoustic-rescue false positives remain outside this fix. No performance claim.Dependencies
None.