Re: [FFmpeg-devel] [PATCH 00/27] Metadata handling in CBS

2021-05-27 Thread Kenny McClive
Hi, Are there any plans to get this (or even a similar patch) merged? I was excited to see this initially, but it has been six months now without a word about it. Thanks, Kenny > On Jan 1, 2021, at 2:35 PM, Mark Thompson wrote: > > Happy New Year! > > The fun part here is: > > AVMastering

Re: [FFmpeg-devel] [PATCH 8/8] RFC: editing HDR properties in H.265 metadata BSF

2020-10-05 Thread Kenny McClive
Hi, Was any additional progress made on this? Forgive me if I’m not finding it. Thanks, Kenny > On Aug 24, 2020, at 5:42 AM, Harry Mallon wrote: > > > > >> On 23 Aug 2020, at 23:33, Mark Thompson wrote: >> >> --- >> Setting HDR properties is a useful feature, but it's very unclear what w

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-11 Thread Kenny McClive
Lynne, Please see my responses below. I am not opposed to taking a different approach, but I could use some guidance on how to go about that. This is my first contribution to FFmpeg, and I struggled to get this done. > For HDR transcoding/tonemapping we rely purely on frame side data atm. How

[FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-10 Thread Kenny McClive
to mean unspecified in other systems such as x265. Signed-off-by: Kenny McClive --- libavcodec/avcodec.h | 90 ++ libavcodec/codec_par.h | 7 ++ libavcodec/options_table.h | 12 +++ libavcodec/utils.c | 134

[FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-10 Thread Kenny McClive
to mean unspecified in other systems such as x265. Signed-off-by: Kenny McClive --- libavcodec/avcodec.h | 90 ++ libavcodec/codec_par.h | 7 ++ libavcodec/options_table.h | 12 +++ libavcodec/utils.c | 134

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-10 Thread Kenny McClive
Sorry, please disregard this patch. I just realized that my test changes are not in it. Kenny > On Aug 10, 2020, at 7:36 PM, Kenny McClive wrote: > > Previously, the only way to input the master display and content light > metadata required for HDR10 was through x265-params. Obv

[FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-10 Thread Kenny McClive
to mean unspecified in other systems such as x265. Signed-off-by: Kenny McClive --- libavcodec/avcodec.h | 84 +++ libavcodec/codec_par.h | 7 ++ libavcodec/options_table.h | 12 libavcodec/utils.c | 133

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-09 Thread Kenny McClive
Thanks, Andreas. When I fix it, do I submit a patch for this patch or a whole new patch with the combined changes? Thanks, Kenny > On Aug 9, 2020, at 4:00 AM, Andreas Rheinhardt > wrote: > > Kenny McClive: >> Hi, >> >> Would someone be willing to help me with

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-08 Thread Kenny McClive
Hi, Would someone be willing to help me with this? This is the first patch I’ve submitted. Patchwork shows that two tests are failing. However, when I run “make fate” locally, I don’t see any failures. Is there some other make or configure command that I can run to make my results match tho

[FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-08 Thread Kenny McClive
to mean unspecified in other systems such as x265. Signed-off-by: Kenny McClive --- libavcodec/avcodec.h | 84 +++ libavcodec/codec_par.h | 7 ++ libavcodec/options_table.h | 12 libavcodec/utils.c | 133