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 750467 - video-color: sRGB matrix is wrongly set to identity
video-color: sRGB matrix is wrongly set to identity
Status: RESOLVED INVALID
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-06-05 20:29 UTC by Nicolas Dufresne (ndufresne)
Modified: 2015-06-12 23:38 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
video-color: Fix definition of sRGB (1.29 KB, patch)
2015-06-05 20:30 UTC, Nicolas Dufresne (ndufresne) 		rejected Details | Review

Description Nicolas Dufresne (ndufresne) 2015-06-05 20:29:05 UTC 
sRGB applies to YUV formats, but for some reason GST defines the color matrix to identity (RGB) which can only apply to RGB format. This cause warning (and failure to expose colorimetry in v4l2) as sRGB is what most webcams uses.
Comment 1 Nicolas Dufresne (ndufresne) 2015-06-05 20:30:13 UTC 
Created attachment 304671 [details] [review]
video-color: Fix definition of sRGB

sRGB is like BT709 but with full range instead. This was oddly
defined with RGB identity matrix instead of BT709 matrix. This
would lead to warning since an RGB matrix was set for YUV formats.
Comment 2 Nicolas Dufresne (ndufresne) 2015-06-08 14:38:32 UTC 
Comment on attachment 304671 [details] [review]
video-color: Fix definition of sRGB

Eh no, that was wrong. Even though all webcams says sRGB they mean sYCC, which was then confirmed by the V4L2 maintainers (in v4l2 they use sRGB not to confuse the driver integrator, they also support a split form of the colorimetry definition).