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 770799 - BT.2020 RGB conversion matrix is wrong
BT.2020 RGB conversion matrix is wrong
Status: RESOLVED FIXED
Product: clutter-gst
Classification: Other
Component: general
3.0.x
Other Linux
: Normal normal
: ---
Assigned To: clutter-gst-maint
clutter-gst-maint
Depends on:
Blocks:
 
 
Reported: 2016-09-03 15:02 UTC by Sebastian Keller
Modified: 2016-09-03 18:56 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
video-sink: Fix colors in UHD videos (1.12 KB, patch)
2016-09-03 15:02 UTC, Sebastian Keller 		committed Details | Review
Comparison with VLC before the patch (936.52 KB, image/png)
2016-09-03 15:06 UTC, Sebastian Keller 		  Details
Comparison with VLC after the patch (944.25 KB, image/png)
2016-09-03 15:07 UTC, Sebastian Keller 		  Details

Description Sebastian Keller 2016-09-03 15:02:49 UTC 
Created attachment 334716 [details] [review]
video-sink: Fix colors in UHD videos

Somebody on reddit noticed that UHD videos have the wrong colors in totem [1]. This seems to be due to the BT.2020 YUV to RGB conversion matrix being wrong. I've recalculated this matrix based on the values found in [2]. Apparently the original matrix just had two positions and signs changed.

[1] https://www.reddit.com/r/Fedora/comments/50vcqy/wrong_colours_in_totem/
[2] http://www.itu.int/rec/R-REC-BT.2020-2-201510-I/en
Comment 1 Sebastian Keller 2016-09-03 15:06:53 UTC 
Created attachment 334717 [details]
Comparison with VLC before the patch
Comment 2 Sebastian Keller 2016-09-03 15:07:30 UTC 
Created attachment 334718 [details]
Comparison with VLC after the patch
Comment 3 Lionel Landwerlin 2016-09-03 18:52:55 UTC 
Review of attachment 334716 [details] [review]:

Looks good!
Thanks a lot for looking into this.
Comment 4 Lionel Landwerlin 2016-09-03 18:56:15 UTC 
Review of attachment 334716 [details] [review]:

Pushed to clutter-gst-3.0 branch.