After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 571147 - [ffmpegcolorspace/videotestsrc] Add support for packed/planar 4:4:4 YUV
[ffmpegcolorspace/videotestsrc] Add support for packed/planar 4:4:4 YUV
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
0.10.x
Other Linux
: Normal enhancement
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-02-10 08:38 UTC by Sebastian Dröge (slomo)
Modified: 2009-02-13 09:11 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Sebastian Dröge (slomo) 2009-02-10 08:38:30 UTC
Hi,
ffmpegcolorspace and videotestsrc currently don't support packed/planar 4:4:4 YUV. Planar 4:4:4 YUV can be output by mpeg2dec as format=Y444 and we can't convert it to anything yet, packed 4:4:4 YUV could be output by mxfdemux at some point and we don't even have a format assigned for it.

So adding support for both is quite easy, question is which format should be used for the packed YUV? We use AYUV for packed 4:4:4:4 YUV, we can't use YUV because it's just 3 characters long... YUVP maybe? fourcc.org also doesn't list anything useful.
Comment 1 Sebastian Dröge (slomo) 2009-02-10 16:46:52 UTC
commit 5fc20b9ec57ad82077d54b4229e8aaa6d9f7aa5b
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Tue Feb 10 17:39:45 2009 +0100

    videotestsrc: Add support for Y444 (planar 4:4:4 YUV)
    
    Partially fixes bug #571147.
Comment 2 David Schleef 2009-02-10 21:00:24 UTC
Is the packed 4:4:4 format 24-bit?  I'm not aware of a fourcc for that.  It would be wise to search for one instead of making one up, though.
Comment 3 David Schleef 2009-02-10 21:03:52 UTC
I *was* aware, but I forgot.  v308

http://developer.apple.com/quicktime/icefloe/dispatch019.html

btw, I have a patches for 10 and 16 bit formats hanging around here somewhere.
Comment 4 Luis de Bethencourt 2009-02-10 23:14:02 UTC
David: can you please share the patches for 10 and 16 bits, they sound very interesting :)
Comment 5 David Schleef 2009-02-10 23:34:45 UTC
It's not especially interesting, since it only works with the colorspace converter in schroedinger and some private schroedinger mods.  But it's converted from CVS now, so it will either get into master or my personal gst tree on fd.o.
Comment 6 Sebastian Dröge (slomo) 2009-02-12 18:09:26 UTC
ffmpegcolorspace already supports conversions from/to Y444. Everything else:

commit 922345af2e13d5464d092761c8401b4c4b716372
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Thu Feb 12 19:02:59 2009 +0100

    ffmpegcolorspace: Add support for packed 4:4:4 YUV (format=V308)
    
    Only conversions from/to are implemented, which
    gives (indirect) support for all possible conversions.
    
    Partially fixes bug #571147.

commit 6dab07f0f67111817dcf3ce05c7ac8be022c5978
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Thu Feb 12 18:17:53 2009 +0100

    videotestsrc: Add support for packed 4:4:4 YUV (format=V308)
    
    Partially fixes bug #571147.
Comment 7 David Schleef 2009-02-12 22:45:09 UTC
that should be lower case v308, btw.
Comment 8 Sebastian Dröge (slomo) 2009-02-13 09:11:50 UTC
(In reply to comment #7)
> that should be lower case v308, btw.

Thanks, fixed in GIT :)