On 29 January 2015 at 09:57, Kieran Kunhya <kier...@obe.tv> wrote: > On 29 January 2015 at 09:20, Paul B Mahol <one...@gmail.com> wrote: >> On 1/29/15, Michael Niedermayer <michae...@gmx.at> wrote: >>> On Wed, Jan 28, 2015 at 03:13:27PM +0000, Paul B Mahol wrote: >>>> Signed-off-by: Paul B Mahol <one...@gmail.com> >>>> --- >>>> 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 ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel