View Bug Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002318 | DCP-o-matic | Bugs | public | 2022-09-04 13:15 | 2023-09-01 21:52 |
Reporter | carl | Assigned To | carl | ||
Priority | urgent | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Target Version | 2.16.31 | ||||
Summary | 0002318: Messed up colours with "HDR" yuv420p10le source | ||||
Description | [] | ||||
Tags | No tags attached. | ||||
Branch | rec2020 (in libdcp and dom) | ||||
Estimated weeks required | |||||
Estimated work required | Medium | ||||
|
Seems OK in VLC and ...later, trying again and looks good in VLC but washed out in |
|
hevc (Main 10), yuv420p10le(tv, bt2020nc/bt2020/smpte2084) |
|
Making a DCP and playing it back in DoM looks closer than ffmpeg but still not a very deep red on the marvel logo. Part of the problem (at least) may be a lack of colourspace conversion / tone mapping for the preview. |
|
also |
|
The main problem here seems to be that for the preview there's no accounting for the difference in primaries between Rec 709 and Rec 2020. Since sRGB and Rec 708 use the same primaries I think we get away with this normally. Putting in a conversion from Rec 2020 to XYZ and back to Rec 709 makes it look a lot better:
Also it looks like there is a different YUV -> RGB conversion for Rec 2020 (I don't understand why) which FFmpeg now supports but libdcp/DoM do not. I haven't managed to see a difference when using that, so either I'm doing it wrong or it's quite subtle. We should also offer tone mapping to account for the fact that Rec 2020 is a wider gamut than DCI P3. FFmpeg has |
|
What needs doing
|
|
Try using |
|
This might be enough for now, it should at least make things a little better. @carl tests running |
|
Merged at |
Date Modified | Username | Field | Change |
---|---|---|---|
2022-09-04 13:15 | carl | New Bug | |
2022-09-04 13:16 | carl | Status | new => confirmed |
2022-09-04 13:16 | carl | Target Version | => 2.16.25 |
2022-09-04 13:16 | carl | Estimated work required | => Undecided |
2022-09-04 13:17 | carl | Note Added: 0005188 | |
2022-09-11 08:37 | carl | Target Version | 2.16.25 => 2.16.26 |
2022-09-18 20:55 | carl | Target Version | 2.16.26 => 2.16.27 |
2022-09-24 21:55 | carl | Target Version | 2.16.27 => 2.16.28 |
2022-10-05 17:48 | carl | Target Version | 2.16.28 => 2.16.30 |
2022-10-05 19:15 | carl | Note Added: 0005254 | |
2022-10-05 19:17 | carl | Note Edited: 0005254 | |
2022-10-05 19:44 | carl | Note Edited: 0005188 | |
2022-10-05 20:03 | carl | Note Added: 0005255 | |
2022-10-05 21:17 | carl | Note Added: 0005256 | |
2022-10-05 22:55 | carl | Note Edited: 0005256 | |
2022-10-05 22:58 | carl | Note Added: 0005257 | |
2022-10-05 23:03 | carl | Note Added: 0005258 | |
2022-10-05 23:10 | carl | Branch | => rec2020 (in libdcp and dom) |
2022-10-05 23:12 | carl | Note Edited: 0005258 | |
2022-10-05 23:13 | carl | Note Edited: 0005258 | |
2022-10-05 23:13 | carl | Note Edited: 0005258 | |
2022-10-05 23:14 | carl | Estimated work required | Undecided => Medium |
2022-10-05 23:21 | carl | Assigned To | => carl |
2022-10-05 23:21 | carl | Status | confirmed => in progress |
2022-10-06 23:45 | carl | Note Added: 0005259 | |
2022-10-12 11:53 | carl | Target Version | 2.16.30 => 2.16.31 |
2022-10-17 23:09 | carl | Note Edited: 0005258 | |
2022-10-17 23:10 | carl | Note Edited: 0005258 | |
2022-10-17 23:25 | carl | Relationship added | related to 0002355 |
2022-10-17 23:25 | carl | Note Added: 0005269 | |
2022-10-18 20:42 | carl | Status | in progress => resolved |
2022-10-18 20:42 | carl | Resolution | open => fixed |
2022-10-18 20:42 | carl | Note Added: 0005270 | |
2022-10-18 20:43 | carl | Note Edited: 0005270 | |
2023-09-01 21:52 | carl | Status | resolved => closed |