Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Paul B Mahol
Dana 15. 10. 2015. 15:44 osoba "Nicholas Robbins" < nickrobbins-at-yahoo@ffmpeg.org> napisala je: > > > On Thursday, October 15, 2015 7:18 AM, Paul B Mahol wrote: > > > > On 10/15/15, Nicolas George wrote: > >> Le quartidi 24 vendemiaire, an CCXXIV, Paul B Mahol a ecrit : > >>> IMHO, it is

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Nicholas Robbins
> On Thursday, October 15, 2015 7:18 AM, Paul B Mahol wrote: > > On 10/15/15, Nicolas George wrote: >> Le quartidi 24 vendemiaire, an CCXXIV, Paul B Mahol a ecrit : >>> IMHO, it is far easier to use this filter than some complicated setpts >>> expression. >> >> True, but shortcuts in the se

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Paul B Mahol
On 10/15/15, Nicolas George wrote: > Le quartidi 24 vendemiaire, an CCXXIV, Paul B Mahol a ecrit : >> IMHO, it is far easier to use this filter than some complicated setpts >> expression. > > True, but shortcuts in the setpts filter would be a better solution for > that: having a bunch of filters

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 5:56 AM, Nicolas George wrote: > Le quartidi 24 vendémiaire, an CCXXIV, Paul B Mahol a écrit : >> IMHO, it is far easier to use this filter than some complicated setpts >> expression. > > True, but shortcuts in the setpts filter would be a better solution for > that: having

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Nicolas George
Le quartidi 24 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > IMHO, it is far easier to use this filter than some complicated setpts > expression. True, but shortcuts in the setpts filter would be a better solution for that: having a bunch of filters that do slightly different things with the PT

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Paul B Mahol
On 10/15/15, Nicolas George wrote: > Le quartidi 24 vendemiaire, an CCXXIV, Paul B Mahol a ecrit : >> This filter sets outlink->frame_rate. Does any of your solutions do that? > > Oh, THAT's what you were talking about! > > Do you realize that this field is purely informational? Having a dedicated

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Nicolas George
Le quartidi 24 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > This filter sets outlink->frame_rate. Does any of your solutions do that? Oh, THAT's what you were talking about! Do you realize that this field is purely informational? Having a dedicated filter to set it seems like utter overkill.

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Paul B Mahol
On 10/15/15, Nicolas George wrote: > Le quartidi 24 vendemiaire, an CCXXIV, Paul B Mahol a ecrit : >> But not framerate. > > Yes, framerate. Please, could you make a little bit of effort so that this > can be a two-sided discussion? > > The following command produces a 40-FPS Matroska file, and th

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Nicolas George
Le quartidi 24 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > But not framerate. Yes, framerate. Please, could you make a little bit of effort so that this can be a two-sided discussion? The following command produces a 40-FPS Matroska file, and the next one transforms it into a 25-FPS file: f

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-15 Thread Paul B Mahol
On 10/15/15, Nicolas George wrote: > Le quartidi 24 vendemiaire, an CCXXIV, Paul B Mahol a ecrit : >> Timebase and stream framerate are same. > > Not at all. For Matroska, timebase is usually 1/1000; for MPEG-[PT]S, > timebase is 1/9; timebase can also be set using settb. But not framerate.

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-14 Thread Nicolas George
Le quartidi 24 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > Timebase and stream framerate are same. Not at all. For Matroska, timebase is usually 1/1000; for MPEG-[PT]S, timebase is 1/9; timebase can also be set using settb. Regards, -- Nicolas George signature.asc Description: Digi

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-14 Thread Paul B Mahol
Dana 14. 10. 2015. 18:50 osoba "Nicolas George" napisala je: > > Le tridi 23 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > > This changes pts but framerate remains same. > > Care to ellaborate? Timebase and stream framerate are same. > > Regards, > > -- > Nicolas George > >

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-14 Thread Nicolas George
Le tridi 23 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > This changes pts but framerate remains same. Care to ellaborate? Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-14 Thread Paul B Mahol
Dana 14. 10. 2015. 16:20 osoba "Nicolas George" napisala je: > > Le primidi 11 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > > >>Ah! So, in other words, it takes input frames, totally disregads their > > >>pts/dts, and puts each one into a cfr output stream at the given fps? > > >> > > >>That s

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-14 Thread Nicolas George
Le primidi 11 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > >>Ah! So, in other words, it takes input frames, totally disregads their > >>pts/dts, and puts each one into a cfr output stream at the given fps? > >> > >>That solves a lot of problems and ridiculous math. :-) I will try it. > > Can't

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-14 Thread Moritz Barsnick
On Tue, Sep 29, 2015 at 23:39:10 +0200, Paul B Mahol wrote: > +static const AVOption setfps_options[] = { > +{ "fps", "A string describing desired output framerate", > OFFSET(framerate), AV_OPT_TYPE_VIDEO_RATE, { .str = "25" }, .flags = V|F }, I know this is taken 1:1 from the fps filter, but

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-12 Thread Paul B Mahol
On 9/29/15, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vf_setfps.c | 119 > +++ > 3 files changed, 121 insertions(+) > create mode 100644 libavfilter/v

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-02 Thread Paul B Mahol
On 10/2/15, Nicholas Robbins wrote: > On Friday, October 2, 2015 2:00 PM, Moritz Barsnick > wrote: >> >> >>On Fri, Oct 02, 2015 at 19:47:12 +0200, Paul B Mahol wrote: >>> fps filter duplicated or removes frames to produce cfr output. >>> This filter changes fps and do not remove or duplicate sing

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-02 Thread Nicholas Robbins
On Friday, October 2, 2015 2:00 PM, Moritz Barsnick wrote: > > >On Fri, Oct 02, 2015 at 19:47:12 +0200, Paul B Mahol wrote: >> fps filter duplicated or removes frames to produce cfr output. >> This filter changes fps and do not remove or duplicate single >> frame so output becomes faster or slower

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-02 Thread Moritz Barsnick
On Fri, Oct 02, 2015 at 19:47:12 +0200, Paul B Mahol wrote: > fps filter duplicated or removes frames to produce cfr output. > This filter changes fps and do not remove or duplicate single > frame so output becomes faster or slower when played when > fps increase or decrease. Ah! So, in other word

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-02 Thread Paul B Mahol
On 10/2/15, Moritz Barsnick wrote: > On Tue, Sep 29, 2015 at 23:39:10 +0200, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavfilter/Makefile | 1 + >> libavfilter/allfilters.c | 1 + >> libavfilter/vf_setfps.c | 119 >> +++ >>

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-10-02 Thread Moritz Barsnick
On Tue, Sep 29, 2015 at 23:39:10 +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vf_setfps.c | 119 > +++ > 3 files changed, 121 insertions(+) > crea

[FFmpeg-devel] [PATCH] avfilter: add setfps filter

2015-09-29 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_setfps.c | 119 +++ 3 files changed, 121 insertions(+) create mode 100644 libavfilter/vf_setfps.c diff --git a/libavfilter/Makefile b/l