Skip to content

Conversation

@Amdrel
Copy link
Collaborator

@Amdrel Amdrel commented Jan 23, 2026

Description

Resolves #45

Currently any videos with subtitles embedded within the video container itself, such as mkv files, do not have their subtitles uploaded when requested. We should import these subtitles. This patch modifies the worker so that it attempts the extraction of subtitles from the video container, and if present, uploads them.

Changes

  • Probe videos for subtitles if they're not detected by yt-dlp and upload them if present.
  • Move the subtitle upload code into a separate function shared by both primary subtitle functions.
  • Add the language_data package to backend/requirements.txt to convert language codes to human-readable descriptions for status updates back to the user.

@don-vip don-vip merged commit d7a3e69 into toolforge:master Jan 23, 2026
1 check 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.

Attempt to extract subtitles from within the file itself

2 participants