Hi

On Fri, Dec 06, 2024 at 03:32:08PM +0100, Niklas Haas wrote:
> From: Niklas Haas <g...@haasn.dev>
> 
> In the long run, it would be ideal if we could add these to the avfilter
> negotiation as well, but for now, this is a good start.
> ---
>  doc/filters.texi       | 56 +++++++++++++++++++++++++++++++
>  libavfilter/vf_scale.c | 75 +++++++++++++++++++++++++++++++++++++++++-
>  2 files changed, 130 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/filters.texi b/doc/filters.texi
> index 9068a8a4e7..eb7463db84 100644
> --- a/doc/filters.texi
> +++ b/doc/filters.texi
> @@ -21189,6 +21189,62 @@ is used by default. Possible values:
>  @item bottom
>  @end table
>  
> +@item in_primaries
> +@item out_primaries
> +Set in/output RGB primaries.
> +
> +This allows the autodetected value to be overridden as well as allows forcing
> +a specific value used for the output and encoder. Possible values:
> +
> +@table @samp
> +@item auto/unknown
> +Choose automatically.
> +
> +@item bt709
> +@item bt470m
> +@item bt470bg
> +@item smpte170m
> +@item smpte240m
> +@item film
> +@item bt2020
> +@item smpte428
> +@item smpte431
> +@item smpte432
> +@item jedec-p22
> +@item ebu3213
> +@end table
> +
> +@item in_transfer
> +@item out_transfer
> +Set in/output transfer response curve (TRC).
> +
> +This allows the autodetected value to be overridden as well as allows forcing
> +a specific value used for the output and encoder. Possible values:
> +
> +@table @samp
> +@item auto/unknown
> +Choose automatically.
> +
> +@item bt709
> +@item bt470m
> +@item gamma22
> +@item bt470bg
> +@item gamma28
> +@item smpte170m
> +@item smpte240m
> +@item linear
> +@item iec61966-2-1
> +@item srgb
> +@item iec61966-2-4
> +@item xvycc
> +@item bt1361e
> +@item bt2020-10
> +@item bt2020-12
> +@item smpte2084
> +@item smpte428
> +@item arib-std-b67
> +@end table
> +

[...]

> @@ -1084,6 +1119,44 @@ static const AVOption scale_options[] = {
>          {"top",           NULL, 0, AV_OPT_TYPE_CONST, 
> {.i64=AVCHROMA_LOC_TOP},         0, 0, FLAGS, .unit = "chroma_loc"},
>          {"bottomleft",    NULL, 0, AV_OPT_TYPE_CONST, 
> {.i64=AVCHROMA_LOC_BOTTOMLEFT},  0, 0, FLAGS, .unit = "chroma_loc"},
>          {"bottom",        NULL, 0, AV_OPT_TYPE_CONST, 
> {.i64=AVCHROMA_LOC_BOTTOM},      0, 0, FLAGS, .unit = "chroma_loc"},
> +    {  "in_primaries", "set input primaries",   OFFSET(in_primaries),  
> AV_OPT_TYPE_INT, { .i64 = -1 }, -1, AVCOL_PRI_NB-1, .flags = FLAGS, .unit = 
> "primaries" },
> +    { "out_primaries", "set output primaries",  OFFSET(out_primaries), 
> AV_OPT_TYPE_INT, { .i64 = AVCOL_PRI_UNSPECIFIED }, 0, AVCOL_PRI_NB-1, .flags 
> = FLAGS, .unit = "primaries"},
> +        {"auto",         NULL,  0, AV_OPT_TYPE_CONST, {.i64=-1},             
>            0, 0, FLAGS, .unit = "primaries"},
> +        {"bt709",        NULL,  0, AV_OPT_TYPE_CONST, 
> {.i64=AVCOL_PRI_BT709},           0, 0, FLAGS, .unit = "primaries"},
> +        {"unknown",      NULL,  0, AV_OPT_TYPE_CONST, 
> {.i64=AVCOL_PRI_UNSPECIFIED},     0, 0, FLAGS, .unit = "primaries"},

please document the meaning difference in the documentation for auto vs 
unspecified

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Take away the freedom of one citizen and you will be jailed, take away
the freedom of all citizens and you will be congratulated by your peers
in Parliament.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to