diff options
| author | Niklas Haas <git@haasn.dev> | 2023-01-21 13:02:36 +0100 |
|---|---|---|
| committer | sfan5 <sfan5@live.de> | 2023-01-24 16:40:37 +0100 |
| commit | b64f9fe646bf929ba7befe13d3413f20fb46b320 (patch) | |
| tree | ed4161ac70becf181a3c4e0c3d2ebdfed9275a36 /video | |
| parent | 3813cc4a14162a901ef636ae5b0914a21702f434 (diff) | |
| download | mpv-b64f9fe646bf929ba7befe13d3413f20fb46b320.tar.bz2 mpv-b64f9fe646bf929ba7befe13d3413f20fb46b320.tar.xz | |
mp_image: fix XYZ primaries default
This was incorrectly set to BT.2020, when it should be DCI-P3 for pretty
much all real-world XYZ content (which is digital cinema content).
Diffstat (limited to 'video')
| -rw-r--r-- | video/mp_image.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/video/mp_image.c b/video/mp_image.c index 52c3f751f2..3bd79b5d0c 100644 --- a/video/mp_image.c +++ b/video/mp_image.c @@ -885,11 +885,11 @@ void mp_image_params_guess_csp(struct mp_image_params *params) // function, but this field can still be relevant for guiding gamut // mapping optimizations, and it's also used by `mp_get_csp_matrix` // when deciding what RGB space to map XYZ to for VOs that don't want - // to directly ingest XYZ into their color pipeline. BT.709 would be a - // sane default here, but it runs the risk of clipping any wide gamut - // content, so we pick BT.2020 instead to be on the safer side. + // to directly ingest XYZ into their color pipeline. We pick DCI-P3 + // because it is the colorspace most closely matching digital cinema + // content, and also has the correct DCI whitepoint. if (params->color.primaries == MP_CSP_PRIM_AUTO) - params->color.primaries = MP_CSP_PRIM_BT_2020; + params->color.primaries = MP_CSP_PRIM_DCI_P3; if (params->color.gamma == MP_CSP_TRC_AUTO) params->color.gamma = MP_CSP_TRC_LINEAR; } else { |