Skip to content

fix: support pyannote 4 diarization API - #39

Merged
FernandoAbishai merged 2 commits into
mainfrom
fix/pyannote-diarization-compat
Aug 23, 2026
Merged

fix: support pyannote 4 diarization API#39
FernandoAbishai merged 2 commits into
mainfrom
fix/pyannote-diarization-compat

Conversation

@FernandoAbishai

Copy link
Copy Markdown
Owner

Context

Phase 5B.6 real-media qualification exposed a pyannote API compatibility
issue in the existing diarization adapter.

The repository permits pyannote.audio 4.x, while the adapter still assumed
the pyannote 3.x authentication and output APIs.

Changes

  • support both token= and legacy use_auth_token= authentication APIs;
  • normalize pyannote 3 Annotation and pyannote 4
    DiarizeOutput.speaker_diarization results;
  • preserve existing overlap-based word/segment speaker assignment;
  • preserve graceful no-token/model-load/runtime failure behavior;
  • redact HF tokens from logs;
  • add focused compatibility smokes.

Validation

  • focused diarization tests: 7/7 PASS
  • backend smoke suite: 100/100 PASS
  • optional-capability smokes: 8/8 PASS
  • python compileall: PASS
  • qualification harness: PASS
  • git diff --check: PASS

Scope

No changes to:

  • requirements
  • qualification matrix
  • transcription engines
  • clip discovery
  • review
  • captions
  • export
  • project schema
  • Electron/runtime
  • release workflows

This is the narrow P1 corrective fix discovered during Phase 5B.6.

Do not merge.

Copilot AI lite review requested due to automatic review settings August 23, 2026 14:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@FernandoAbishai
FernandoAbishai merged commit 67db48d into main Aug 23, 2026
2 checks passed
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