[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