[mythtv] YV12 problem

Jean-Yves Avenard jyavenard at gmail.com
Wed Dec 12 07:51:56 UTC 2018


YV12 (same as yuv420p) is 3 planes, nv12 is 2 planes (u and v planes interleaved). 

Modifying the OGL deinterlacer to handle nv12 should be trivial. I don't know of any hardware decoder outputting 3 planes yuv. 

Next month I'm back in Oz permanently where I have a working mythtv system. I'll be able to get back into coding a bit (plus I need a new hobby). 
Happy to give a hand then. 

Get Firefox on Android
________________________________
From: Peter Bennett
Sent: Tuesday, 11 December 2018 23:27
To: David Engel; Development of MythTV
Subject: Re: [mythtv] YV12 problem

On 12/11/18 3:13 PM, Peter Bennett wrote:
> Hi David
>
> Please can you point me to the patches that you are planning to 
> install, or specific instructions on which commits you will reverse, 
> so that I can do a test.
>
> Here are close-up photos of the screen from current master with UYVY 
> and YV12 when using fire stick (non-4k) or "mi box s" or other devices 
> that do not support high precision opengl es.
>
> Screen using YV12 checked ---> https://imgur.com/lbfyEWQ
> Screen using YV12 unchecked (i.e. using UYVY) ---> 
> https://imgur.com/dLoMUau
>
> If you send me your patches I will test with those and see how the 
> picture looks.
>
> Peter

I have tried your patch on ticket #13358. It produces the jagged result 
(https://imgur.com/dLoMUau) on the Fire TV (non-4k), without any ability 
to fix it.

I agree that the code you are removing is flawed, as stated by Mark. I 
recommend that we change the default for the "enable YV12" setting  to 
be "false" for everybody, which bypasses that flawed code and achieves 
the same effect as removing it. The users of fire tv non-4k, mi box s, 
or other android devices that do not support high precision can still 
enable that check box to fix the jaggies, albeit with flawed code that 
does not properly support kernel deinterlace. However, using Linear 
blend deinterlace with YV12 is better than looking at that jagged picture.

If somebody can come up with better code that supports these android 
devices without the jaggies, we can then remove that faulty YV12 code.

If my recommendation is not taken and the code is removed, then we need 
to remove the setting from the frontend setup for enabling/disabling 
YV12, since that will no longer have any effect. I will also need to 
update the wiki to remove the recommendations around YV12 for android.

Peter
_______________________________________________
mythtv-dev mailing list
mythtv-dev at mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mythtv.org/pipermail/mythtv-dev/attachments/20181212/a3d53142/attachment.html>


More information about the mythtv-dev mailing list