search

unclenoriega / Lazyman.bundle

Lazyman Plex Channel
11 stars 4 forks source link

Select best stream /w 60fps supported #1

Closed bitmarshal closed 7 years ago

bitmarshal commented 7 years ago

This covers two issues I ran into.

Issue 1. The pattern match ignored streams with the frame rate specified which skipped the 59.94 fps streams.

Issue 2. Plex seemed to always consider the first stream added to the objects array as the original source even if it was of lower resolution preventing the use of a higher quality stream.

unclenoriega commented 7 years ago

Issue 1. This is something I had wanted to fix. If I understand it correctly, this can be accomplished simply by changing the regular expression to your new one.

Issue 2. I'm not sure this is the case. Or maybe it depends on the Plex client? I'm seeing this on the web interface—it chooses the first one listed to play—but on the Android client, it's choosing 720p regardless of the order of the source videos.

I want to include 60 fps streams, but I would rather continue to show all of the available versions instead of only the best one (even though the latest update seems to have screwed up the "Play version" option for HLS streams on the web player).

Can you change it so that it adds all available stream versions? If you're worried about "Issue 2", maybe sort them before adding? I don't think this should be necessary, but some of the plex clients seem to be buggy at the moment.

bitmarshal commented 7 years ago

Changed it to keep all the streams with the highest quality as first in the list.

unclenoriega commented 7 years ago

Looks good. Thanks!