[FFmpeg-devel] [PATCH] avcodec/dpxenc: support colour metadata in DPX encoder, fixes ticket #6023

[Kieran O Leary]

Hi

On 2 Feb 2017 08:20, "Carl Eugen Hoyos" <ceffmpeg at gmail.com> wrote:

2017-02-01 23:37 GMT+01:00 Kieran O Leary <kieran.o.leary at gmail.com>:

> DPX contains some values that are super-specific to film, such as
'Printing
> Density'. I do not see that value in either of those enums but it's a
> common value that pops up in film scans.

Then I believe you have to add this property to the enums.


Will do. In a separate patch?


> The larger issue that I see with this approach is that it's an integer
that
> needs to be written to the 801/802 offsets in the DPX header. So if I want
> logarithmic transfer characteristic, 3 needs to be written as the value
and
> this is interpreted by the decoder. Logarithmic is listed as 9 in
>  AVColorTransferCharacteristic

This means you need a translation in dpxenc.c.


Ah,  great. Thanks for the heads up. I'll prepare another patch once I
figure this out.

Thanks for your help!

-Kieran