From b64f9fe646bf929ba7befe13d3413f20fb46b320 Mon Sep 17 00:00:00 2001 From: Niklas Haas Date: Sat, 21 Jan 2023 13:02:36 +0100 Subject: [PATCH] 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). --- video/mp_image.c | 8 ++++---- 1 file 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 {