On 29 January 2015 at 09:57, Kieran Kunhya <[email protected]> wrote: > On 29 January 2015 at 09:20, Paul B Mahol <[email protected]> wrote: >> On 1/29/15, Michael Niedermayer <[email protected]> wrote: >>> On Wed, Jan 28, 2015 at 03:13:27PM +0000, Paul B Mahol wrote: >>>> Signed-off-by: Paul B Mahol <[email protected]> >>>> --- >>>> Bit-exact with mp=softpulldown except first frame which is uninitialized >>>> data in mp filter. >>> >>>> TODO: pts needs update, workaround is to use setpts filter. >>> >>> patch LGTM except the pts / time_base issue >> >> mp=softpulldown sets all frames to MP_NOPTS_VALUE. >> >> I do not see way how to set filter to use variable frame rate, which >> is possible as repeat_pict can change during stream. > > IMO the filter needs to reject a timebase that's not 1001/24000 > exactly and it'll need to change the timebase and pts to 1001/30000. > See how x264 does it. > > Kieran
Actually, the PTS of an encoder with these repeat flags is actually the PTS *after* pulldown. Kieran _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
