[FFmpeg-devel] [PATCH] Add YCOCG colorspace to input arguments of vf_colorspace.

[Ronald S. Bultje]

Hi,

On Thu, May 19, 2022 at 4:42 PM Mohammad Izadi <
izadi-at-google.com at ffmpeg.org> wrote:

> ffmpeg support YCOCG (YCOCG=YCGCO). However, vf_colorspace is only support
> YCGCO as input. Added YCOCG to the inputs.
> ---
>  libavfilter/vf_colorspace.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c
> index 3c8b3b20eb..bccd04528e 100644
> --- a/libavfilter/vf_colorspace.c
> +++ b/libavfilter/vf_colorspace.c
> @@ -906,6 +906,7 @@ static const AVOption colorspace_options[] = {
>      ENUM("smpte170m",   AVCOL_SPC_SMPTE170M,   "csp"),
>      ENUM("smpte240m",   AVCOL_SPC_SMPTE240M,   "csp"),
>      ENUM("ycgco",       AVCOL_SPC_YCGCO,       "csp"),
> +    ENUM("ycocg",       AVCOL_SPC_YCOCG,       "csp"),
>      ENUM("gbr",         AVCOL_SPC_RGB,         "csp"),
>      ENUM("bt2020nc",    AVCOL_SPC_BT2020_NCL,  "csp"),
>      ENUM("bt2020ncl",   AVCOL_SPC_BT2020_NCL,  "csp"),
> --
> 2.36.1.124.g0e6072fb45-goog
>

Could you elaborate on why you think AVCOL_SPC_YCGCO might be missing from
the LumaCoefficients table? Is this a typo/mixup? Should it be added?

Ronald