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

Playback pauses when turning off streamable subtitles (but not when turning them on!) #1628

Closed
dathbe opened this issue Jan 9, 2024 · 9 comments
Labels
bug Something isn't working

Comments

@dathbe
Copy link

dathbe commented Jan 9, 2024

Software Versions

  • Jellyfin Server Version: 10.8.13
  • Roku Client Version: 2.0.1 (2.0 build 1)

Describe the bug

When watching a video and text (SRT) subtitles are turned on, the video playback continues uninterrupted.

But when the subtitles are then turned off again, the playback pauses for a second and then resumes--much like what happens when graphic subtitles are turned on/off and the server has to start or stop an transcode.

The app reports it is Direct Playing the video file when the subtitles are on and off.

How To Reproduce

  1. Play a movie with SRT subtitles
  2. While watching the movie turn the subtitles on (using the up button options menu)
  3. While still watching the movie, turn the subtitles off (same way)
  4. Bug occurs

Expected behavior

Playback should not be interrupted when turning on or off streamable subtitles

Logs

N/A

Screenshots

N/A

Connection Information

  • Is server local or remote? Local
  • Is server connection HTTP or HTTPS? HTTPS

Additional context

This may be related to #1627 where the app seems to pick up SRT subtitles as graphical subtitles.

@dathbe dathbe added the bug Something isn't working label Jan 9, 2024
@ebits21
Copy link

ebits21 commented Jan 10, 2024

I've noticed this as well.

@1hitsong
Copy link
Member

This may be related to #1627 where the app seems to pick up SRT subtitles as graphical subtitles.

It's def related. The stop/start occurs when we have to kick off a new content stream. We have code to only do that if the subtitles are not text.

Can you cut a 10 second sample of your file with the streams for me to test with? I've exhausted my subtitles test file and non exhibit this behavior.

@dathbe
Copy link
Author

dathbe commented Jan 11, 2024

Sent you an invite to a private repository with a test file that is exhibiting the behavior on my setup. File is called demo.mkv.

@1hitsong
Copy link
Member

File received. I'll check it out.

On a side note, I've seen this opening clip (and the rest of the movie) so many times I could recreate it frame by frame from memory. 😆

@1hitsong
Copy link
Member

I think I figured out the path to the bug.

@1hitsong
Copy link
Member

@dathbe If you can, please give this a try. It fixed the video stop/start when turning off text subtitles for me in my testing.

@dathbe
Copy link
Author

dathbe commented Jan 12, 2024

Confirmed that PR #1632 fixes this issue for me. I assume that neither 1632 nor 1633 incorporates the other because 1632 does not fix issue 1627 and 1633 does not fix this issue. I imagine the next production release will include both, but I wanted to flag it in case not.

@1hitsong
Copy link
Member

Correct. PRs are self contained. These 2 bug fixes won't exist together until they get merged into the 2.0.x branch.

@1hitsong
Copy link
Member

Fixed by #1632

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants