Skip to content

Fix: WhisperAI fails on files with multiple audio streams of same lan…#3330

Open
Brennmeister1 wants to merge 6 commits into
morpheus65535:masterfrom
Brennmeister1:fix/whisper-multi-audio-stream
Open

Fix: WhisperAI fails on files with multiple audio streams of same lan…#3330
Brennmeister1 wants to merge 6 commits into
morpheus65535:masterfrom
Brennmeister1:fix/whisper-multi-audio-stream

Conversation

@Brennmeister1
Copy link
Copy Markdown

…guage

  • Changed ffmpeg map from '0:a:m:language:X' to '0:a:language:X'
  • The Ⓜ️ (multiple) flag caused ffmpeg to select ALL audio streams matching the language, but s16le format only supports one audio stream
  • Now selects only the first matching stream (usually the correct one)

…guage

- Changed ffmpeg map from '0:a:m:language:X' to '0:a:language:X'
- The :m: (multiple) flag caused ffmpeg to select ALL audio streams matching
  the language, but s16le format only supports one audio stream
- Now selects only the first matching stream (usually the correct one)
- Dockerfile builds on top of lscr.io/linuxserver/bazarr with patched whisperai.py
- GitHub Actions workflow builds and pushes to ghcr.io on branch push
- Image tagged as ghcr.io/brennmeister1/bazarr:fix-multi-audio
Copy link
Copy Markdown
Owner

@morpheus65535 morpheus65535 left a comment

Choose a reason for hiding this comment

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

PR must be based on development branch, not master. You should also include only whisperai.py, not the other files.

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