Skip to content

[pull] main from m-bain:main#70

Merged
pull[bot] merged 6 commits intoAsofwar:mainfrom
m-bain:main
Mar 25, 2026
Merged

[pull] main from m-bain:main#70
pull[bot] merged 6 commits intoAsofwar:mainfrom
m-bain:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Mar 25, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

claude and others added 6 commits March 25, 2026 08:31
Characters not in the alignment model's vocabulary (digits, symbols,
foreign script) were excluded from CTC alignment and got no timestamps.

Fix: extend the emission matrix with a wildcard column (max non-blank
score per frame) so unknown chars participate in alignment via the
existing get_trellis/backtrack — no changes to the CTC internals.
Interpolation fallback fills any remaining gaps as a safety net.

Unlike PR #986, this does not rewrite get_trellis or backtrack, so it
avoids the regression that caused #1220.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Reproduces the exact reported bug: '4,9' embedded in text gets no
start/end/score because digits and commas are not in the alignment
model dictionary.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add pytest to optional dev dependencies and create a dedicated
tests.yml workflow to run pytest across Python 3.10-3.13.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@pull pull bot locked and limited conversation to collaborators Mar 25, 2026
@pull pull bot added the ⤵️ pull label Mar 25, 2026
@pull pull bot merged commit 095b36b into Asofwar:main Mar 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants