[FFmpeg-devel] [FFMPEG DEVEL] [PATCH v4] fftools/ffprobe: Add S12M Timecode output as side data (such as SEI TC)

[Antonin Gouzer]

Hello sorry for the delay.
Yes you're right.
There is 3 timecodes data for one frame.
One SEI data with 3 timecodes would represented by 3 side data.
As far as I know there is only one timecode in the SEI data.
In order to keep coherence with the schema (No simple element) , we could
implement  this:

            <side_data_list>
                <side_data side_data_type="SMPTE 12-1 timecode">
                 <timecode value="00:03:26:03"/>
                 <timecode value="00:03:26:03"/>
                 <timecode value="10:00:25:03"/>
               </side_data>
            </side_data_list>
        </frame>

Would it be correct ?

Le dim. 19 mai 2019 à 19:50, Marton Balint <cus at passwd.hu> a écrit :

>
>
> On Sun, 19 May 2019, Antonin Gouzer wrote:
>
> > ---
> > Add the index of the timecode in case of multiple timecodes values
> > Limit to 3 the number of timecodes
> > remove break
>
> Am I reading this correctly, you'd get XML like
>
> <side_data side_data_type="SMPTE 12-1 timecode" timecode="00:03:25:03"
> id="1"/>
> <side_data side_data_type="SMPTE 12-1 timecode" timecode="00:03:25:03"
> id="2"/>
> <side_data side_data_type="SMPTE 12-1 timecode" timecode="00:03:25:03"
> id="3"/>
>
> right? The problem is that your output now looks as if the frame have 3
> instances of SMPTE 12-1 timecode side data.
>
> Maybe it's just me, but I'd prefer something like
>
> <side_data side_data_type="SMPTE 12-1 timecode">
> <timecode>00:03:25:03</timecode>
> <timecode>00:03:25:03</timecode>
> <timecode>00:03:25:03</timecode>
> </side_data>
>
> What do you think?
>
> Thanks,
> Marton
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".