Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(HLS): Fix subtitle timing #7625

Merged
merged 1 commit into from
Nov 19, 2024
Merged

Conversation

avelad
Copy link
Member

@avelad avelad commented Nov 19, 2024

Simplifies subtitle management for sequence mode and segments mode
Runs subtitle tests in segments mode as well (Tizen 3.0)

Fixes: #7447

@avelad avelad added type: bug Something isn't working correctly component: HLS The issue involves Apple's HLS manifest format priority: P1 Big impact or workaround impractical; resolve before feature release labels Nov 19, 2024
@avelad avelad force-pushed the hls-subtitles branch 2 times, most recently from 51ee777 to a21aedd Compare November 19, 2024 11:36
@avelad
Copy link
Member Author

avelad commented Nov 19, 2024

@shaka-bot test

@shaka-bot
Copy link
Collaborator

@avelad: Lab tests started with arguments:

  • pr=7625

Simplifies subtitle management for sequence mode and segments mode
Runs subtitle tests in segments mode as well
@avelad avelad added this to the v4.13 milestone Nov 19, 2024
@avelad avelad merged commit 9e2b15e into shaka-project:main Nov 19, 2024
37 checks passed
@avelad avelad deleted the hls-subtitles branch November 19, 2024 17:37
joeyparrish pushed a commit that referenced this pull request Nov 19, 2024
Simplifies subtitle management for sequence mode and segments mode 
Runs subtitle tests in segments mode as well (Tizen 3.0)

Fixes: #7447
joeyparrish pushed a commit that referenced this pull request Nov 19, 2024
Simplifies subtitle management for sequence mode and segments mode 
Runs subtitle tests in segments mode as well (Tizen 3.0)

Fixes: #7447
joeyparrish pushed a commit that referenced this pull request Nov 19, 2024
Simplifies subtitle management for sequence mode and segments mode 
Runs subtitle tests in segments mode as well (Tizen 3.0)

Fixes: #7447
@markuswolters
Copy link

This fix seems to introduce a new bug 4.12.2/4.11.14 introduces subtitles with HLS streams being out of sync

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: HLS The issue involves Apple's HLS manifest format priority: P1 Big impact or workaround impractical; resolve before feature release type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Subtitles in HLS are shown too early
4 participants