[FFmpeg-devel] [PATCH] Add 'Presentation Y offset' metadata

[tomas.hardin at codemill.se]

On 2015-03-30 14:21, Michael Niedermayer wrote:
> On Mon, Mar 30, 2015 at 12:31:04PM +0200, tomas.hardin at codemill.se 
> wrote:
>> On 2015-03-30 12:19, tim nicholson wrote:
>> >Some software mis decodes IMX material in mxf if this metadata field is
>> >missing. See the discussion on the ffmpeg ML:-
>> >
>> >"[FFmpeg-user] How to set 3 specific metadata flags
>> >(ITU601/displayoffset) in FFmpegs IMX50 MXF-OP1a encoding?"
>> >
>> >This patch add this field to mxf wrapped material.
>> 
>> Looks fine, but I know from experience that all this stuff gets
>> tricky quickly. Like there should be mechanisms in lavf for dealing
>> with presentation rectangles and all that jazz, including different
>> aspect ratios at different layers.
> 
> should i apply this patch or wait ?

Apply it - dealing with this properly (like for properly remuxing MOV 
<--> MXF) would require quite a bit of design that shouldn't hold up 
this patch.

/Tomas