Skip to content

Add FilterCorpusByDiarizationJob - #669

Open
robin-p-schmitt wants to merge 5 commits into
mainfrom
robin-filter-by-diarization
Open

robin-p-schmitt wants to merge 5 commits into
mainfrom
robin-filter-by-diarization

Conversation

@robin-p-schmitt

Copy link
Copy Markdown
Contributor

No description provided.

curufinwe and others added 4 commits November 4, 2026 14:42
parse_rttm returns frozen dataclasses with all fields of an RTTM SPEAKER
line (<NA> as None) instead of (speaker, start, end) tuples, so the file
id and channel are no longer dropped. Postponed annotations keep the
module importable on older Pythons.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Fold the speaker_changes helper into run() and define in the docstring
how this job counts a speaker change. Drops the helper from __all__ and
the module-level Sequence[tuple[...]] annotation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Runs as a mini task unless rqmt is set. rqmt is not a constructor
parameter, so the hash is unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Reads rttm_dir/<recording.fullname()>.rttm, so recordings with the same
name in different subcorpora no longer share an RTTM, and reports the
full name in skipped.jsonl. Clarifies the corpus and rttm_dir docstrings.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@curufinwe curufinwe changed the title corpus filter: add FilterCorpusByDiarizationJob Add FilterCorpusByDiarizationJob Nov 4, 2026
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.

2 participants