Jump to content


Photo

Theater mpv test


  • Please log in to reply
3877 replies to this topic

#3861 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted 23 April 2019 - 09:57 AM

I didn't test live TV with the latest MPV git builds. Did it work for you, with that?

#3862 markyp OFFLINE  

markyp

    Advanced Member

  • Members
  • 233 posts
  • Local time: 10:08 AM

Posted 23 April 2019 - 10:01 AM

I didn't test live TV with the latest MPV git builds. Did it work for you, with that?


I haven’t updated mpv over the top yet, if that’s what you mean?

#3863 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted 23 April 2019 - 10:14 AM

I haven’t updated mpv over the top yet, if that’s what you mean?


Yeah, that's what causes the live TV issue.

#3864 daedalus OFFLINE  

daedalus

    Advanced Member

  • Members
  • 894 posts
  • Local time: 12:08 PM

Posted 23 April 2019 - 10:29 AM

desktop still "shines" through for a second on transition to the next episode

episode1 ends -> desktop visible -> default backdrop visible -> episode2 starts

 

description text is not changing, warning on audio passthrough page about this option and/or hiding resample option if passthrough is ticked, is missing and you should use the official names of the settings

 

both still present in latest test build, along with the still present "blue radiance" background on every app start, no matter what theme is selected

 

changing the app theme in ET don't survive restarting the app, it always loads the one set before the testbuild



#3865 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted 23 April 2019 - 11:09 PM

 

I just tried the latest git build, and that also had the Live TV issue. That commit I referred to, is the only thing that I can see that might be the issue. But, in the playbackhandler, maybe this section?

if (options.largeCache) {

        list.push('--demuxer-readahead-secs=1800');
        list.push('--cache-secs=1800');

        var cacheSize = 2097152;
        var backBuffer = Math.round(cacheSize * .8);
        list.push('--cache=' + cacheSize.toString());
        list.push('--cache-backbuffer=' + backBuffer.toString());
        list.push('--force-seekable=yes');
        list.push('--hr-seek=yes');
        //list.push('--demuxer-lavf-hacks=no');

 

Well, it doesn't appear to be this. I deleted all of that, and live tv still won't start :/



#3866 Luke OFFLINE  

Luke

    System Architect

  • Administrators
  • 124564 posts
  • Local time: 06:08 AM

Posted Yesterday, 01:23 PM

I've updated the build in the first post and tagged the version as 3.0.7. It has the updated electron 5.0 final release. Thanks.



#3867 daedalus OFFLINE  

daedalus

    Advanced Member

  • Members
  • 894 posts
  • Local time: 12:08 PM

Posted Yesterday, 03:39 PM

everything mentioned in my last post still present in latest build

along with the gpu-api=opengl "transparent video bug", but i gave up on that <_<



#3868 Luke OFFLINE  

Luke

    System Architect

  • Administrators
  • 124564 posts
  • Local time: 06:08 AM

Posted Yesterday, 03:47 PM

 

 

along with the gpu-api=opengl "transparent video bug

This has already been previously discussed, right? You're not going to be able to do this right now.



#3869 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted Yesterday, 10:48 PM

Running the new build, now. Looks good, so far. Everything seems to work ok. The video coloring is different to the git builds, but I've mentioned that, before. It seems brighter. And the description for video sync still needs to be changed. As is, it only applies to one option. And sadly, I have to hack the playbackhandler again, for hwdec :/ If you can change the 'no' option to be vacant, maybe that will yield to the mpv.conf



#3870 Luke OFFLINE  

Luke

    System Architect

  • Administrators
  • 124564 posts
  • Local time: 06:08 AM

Posted Today, 01:02 AM

Well i was able to get newer builds of mpv working, but I don't think we're going to use them at this time. They've removed the stream caching feature, and I'm unable to get live stream seeking to work without this. We'd have to implement it differently, potentially outside of mpv, which is fine if that were required.

 

But now, looking at this, it seems they are re-thinking it, and a huge amount of changes are potentially going to be rolled back:

https://github.com/m...r/mpv/pull/6659

 

I think for now at least, I don't have a high confidence level of using the newer builds.



#3871 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted Today, 01:08 AM

Can you use stream cache via lavfi?



#3872 Luke OFFLINE  

Luke

    System Architect

  • Administrators
  • 124564 posts
  • Local time: 06:08 AM

Posted Today, 01:10 AM

Possibly, but knowing all of those changes that are probably about to get merged in, I'm not really sure we should put newer mpv builds into production until that whole situation is settled.



#3873 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted Today, 01:12 AM

Also, the git build from March 10th, is the build I've been using in Theater. Live TV still works with that, and the color mapping is better than the build you've using in the test builds of Theater.



#3874 Luke OFFLINE  

Luke

    System Architect

  • Administrators
  • 124564 posts
  • Local time: 06:08 AM

Posted Today, 01:14 AM

In any case I've updated the build in the first post again:

  • Added an unset option for hwa which is what i think you're looking for
  • The auto option for deinterlace will apply deinterlace=yes if the current video has been detected as interlaced by the server. This will be the suggested approach because mpv docs say that applying deinterlace=yes when not needed will make things worse.


#3875 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted Today, 01:29 AM

 

In any case I've updated the build in the first post again:

  • Added an unset option for hwa which is what i think you're looking for
  • The auto option for deinterlace will apply deinterlace=yes if the current video has been detected as interlaced by the server. This will be the suggested approach because mpv docs say that applying deinterlace=yes when not needed will make things worse.

 

 

 

Looks like you've finally got that nailed down. Using auto deinterlacing now appears to work, and the hwdec=no works, as long as you don't enable other options like deinterlacing, which will then disable hwdec. But I can work with that.



#3876 Luke OFFLINE  

Luke

    System Architect

  • Administrators
  • 124564 posts
  • Local time: 06:08 AM

Posted Today, 01:31 AM

Ok, I'm going to release this as 3.0.8. Once mpv makes a decision and moves forward on stream caching or not, and then all of that stabilizes, then we can do another release with mpv updated.



#3877 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted Today, 01:34 AM

You still need to change the description for video sync. 



#3878 Doofus OFFLINE  

Doofus

    Advanced Member

  • Members
  • 11018 posts
  • Local time: 03:08 AM

Posted Today, 02:16 AM

It's a pity the color mapping didn't get corrected. It's pretty bad. The saturation is noticeably less. But at least I can still use the git build from March 10th.






0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users