Skip to content

fix(sortformer): honor packed streaming state - #137

Merged
ivan-digital merged 1 commit into
mainfrom
fix/sortformer-packed-stream-state
Aug 15, 2026
Merged

fix(sortformer): honor packed streaming state#137
ivan-digital merged 1 commit into
mainfrom
fix/sortformer-packed-stream-state

Conversation

@ivan-digital

Copy link
Copy Markdown
Member

Summary

  • start Sortformer's valid speaker-cache prefix at zero and pass its real length to the ONNX graph
  • read the export's packed prediction prefix instead of fixed cache/FIFO capacity offsets
  • refresh pre-compression cache predictions to match NeMo's reference update
  • update regression coverage and wrapper documentation

Evidence

  • clean default build: 31/31 CTest tests pass
  • ONNX targets build; Sortformer cache test passes
  • 12-file Azerbaijani benchmark: corrected ONNX path 26.30% DER / 100% count agreement versus deployed API 57.20% / 83.3%; direct NeMo v2.1 is 27.06% / 100%
  • on a representative recording, the corrected path removes the label permutation at the first 27.2-second boundary and matches NeMo's timeline with zero speaker confusion

@ivan-digital
ivan-digital merged commit ec99c1b into main Aug 15, 2026
13 checks passed
@ivan-digital
ivan-digital deleted the fix/sortformer-packed-stream-state branch August 15, 2026 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant