Skip to content

Support known speaker hints in REST API#506

Merged
boxerab merged 1 commit into
collabora:mainfrom
boxerab:fix-rest-known-speakers
Jun 14, 2026
Merged

Support known speaker hints in REST API#506
boxerab merged 1 commit into
collabora:mainfrom
boxerab:fix-rest-known-speakers

Conversation

@boxerab

@boxerab boxerab commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • support OpenAI-compatible REST known speaker fields instead of warning that they are ignored
  • allow multipart known_speaker_references uploads to enroll named speakers with the existing diarizer
  • include speaker labels on verbose_json segments when REST speaker hints are provided
  • document REST known speaker fields and add diarizer/helper coverage

Tests

  • ruff check README.md tests/test_diarization.py tests/test_server_extended.py whisper_live/diarization.py whisper_live/server.py
  • python -m py_compile whisper_live/diarization.py whisper_live/server.py tests/test_diarization.py tests/test_server_extended.py
  • PYTHONPATH="$tmpdir:$PWD" pytest tests/test_diarization.py tests/test_server_extended.py -q with a temporary local torch stub because this workstation does not have torch installed on system python: 69 passed

Notes

  • Full ruff check . still reports pre-existing lint issues outside this change.
  • black --check on the touched files would reformat unrelated existing code in server.py and tests/test_server_extended.py, so this PR avoids that formatting churn.

@boxerab boxerab requested a review from zoq June 4, 2026 20:05
@boxerab boxerab force-pushed the fix-rest-known-speakers branch from 125a02c to ea97036 Compare June 4, 2026 20:29
@boxerab

boxerab commented Jun 14, 2026

Copy link
Copy Markdown
Collaborator Author

As all tests are passing and this is needed by Aavaaz project, going to merge.

@boxerab boxerab merged commit 44940e2 into collabora:main Jun 14, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants