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

Remove HEVC SDR fallback stream #10751

Closed

Conversation

dmitrylyzo
Copy link
Contributor

@dmitrylyzo dmitrylyzo commented Dec 20, 2023

Alternative to #9121

The client starts playback from a stream with an average bitrate. In some cases this is HEVC SDR. After a while, the client switches to the HDR stream, but the server doesn't restart ffmpeg and the client continues playing the old stream.

Since we have a VideoRange condition, the HEVC SDR fallback stream is no longer needed, IMO.

Alternatively, we can use bitrate copying - #9121

Changes
Remove HEVC SDR fallback stream.

Issues
Fixes #8043
Fixes jellyfin/jellyfin-webos#105
Fixes jellyfin/jellyfin-tizen#163
Fixes jellyfin/jellyfin-webos#59
Fixes #5576 (#5576 (comment))
Fixes jellyfin/jellyfin-web#1190

The client starts playback from a stream with an average bitrate.
In some cases this is HEVC SDR. After a while, the client switches
to the HDR stream, but the server doesn't restart ffmpeg and
the client continues playing the old stream.

Since we have a VideoRange condition, the HEVC SDR fallback stream
is no longer needed.
@nyanmisaka
Copy link
Member

Is there a way to check the client name and bypass this?
AFAIK Apple is still very picky about some stream-copied HDR clips even if HDR is definitely supported by the device.

41BA6084-434C-4C05-A6D2-50BA93B09945

@dmitrylyzo
Copy link
Contributor Author

Is there a way to check the client name and bypass this?

I don't think we should use the client name to workaround this issue. Instead, we need to add some condition to the device profile.

AFAIK Apple is still very picky about some stream-copied HDR clips even if HDR is definitely supported by the device.

There isn't much information in the playlist.
Does it just try each stream until success?

#9121 is a softer solution.

@jellyfin-bot jellyfin-bot added the merge conflict Merge conflicts should be resolved before a merge label Feb 12, 2024
@jellyfin-bot
Copy link
Contributor

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@dmitrylyzo
Copy link
Contributor Author

Superseded by #9121

@dmitrylyzo dmitrylyzo closed this Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge conflict Merge conflicts should be resolved before a merge
Projects
None yet
3 participants