Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-27 Thread Michael Niedermayer
On Fri, Mar 27, 2015 at 11:07:18AM +0530, Himangi Saraogi wrote: --- Changelog | 1 + doc/filters.texi| 27 libavfilter/Makefile| 1 + libavfilter/allfilters.c| 1 + libavfilter/vf_detelecine.c | 345

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-27 Thread Paul B Mahol
Dana 27. 3. 2015. 15:07 osoba Himangi Saraogi himangi...@gmail.com napisala je: --- Changelog | 1 + doc/filters.texi| 27 libavfilter/Makefile| 1 + libavfilter/allfilters.c| 1 + libavfilter/vf_detelecine.c | 346

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-27 Thread Michael Niedermayer
On Fri, Mar 27, 2015 at 07:36:05PM +0530, Himangi Saraogi wrote: --- Changelog | 1 + doc/filters.texi| 27 libavfilter/Makefile| 1 + libavfilter/allfilters.c| 1 + libavfilter/vf_detelecine.c | 346

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-27 Thread Himangi Saraogi
On 27 March 2015 at 22:00, Paul B Mahol one...@gmail.com wrote: Dana 27. 3. 2015. 15:07 osoba Himangi Saraogi himangi...@gmail.com napisala je: --- Changelog | 1 + doc/filters.texi| 27 libavfilter/Makefile| 1 +

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-26 Thread Michael Niedermayer
On Thu, Mar 26, 2015 at 01:54:44PM +0530, Himangi Saraogi wrote: --- Changelog | 1 + doc/filters.texi| 27 libavfilter/Makefile| 1 + libavfilter/allfilters.c| 1 + libavfilter/vf_detelecine.c | 344

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-25 Thread Michael Niedermayer
On Wed, Mar 25, 2015 at 03:38:18PM +0530, Himangi Saraogi wrote: This is an exact inverse of the telecine filter unlike previously existing pullup and fieldmatch ones. Tested on few samples generated using the telecine filter. Documentation is yet to be added. Added an additional parameter

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-25 Thread Carl Eugen Hoyos
Himangi Saraogi himangi774 at gmail.com writes: +++ b/Changelog - libdcadec wrapper - Drop support for nvenc API before 5.0 - nvenc H265 encoder - +- Detelecine filter Please do not add the empty line. Add a short documentation that emphasizes that the new filter only works with a

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-23 Thread Kieran Kunhya
Kieran, the material used in testing is CFR to the best of my knowedge the material does not start at pts 0, the code ignores the start time aka the first pts. Theres only one possibility to fix this and that is to use the first pts, no (de)telecine operation can maintain sync with another

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-23 Thread Michael Niedermayer
On Mon, Mar 23, 2015 at 11:54:17AM +, Kieran Kunhya wrote: why does the filter not use the input pts? Michael, none of the PTS hacks will ever work - (inverse)telecine is a CFR operation. It inherently requires knowing the duration of the frame and in VFR mode you don't know it.

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-23 Thread Michael Niedermayer
On Mon, Mar 23, 2015 at 12:47:54PM +, Kieran Kunhya wrote: Kieran, the material used in testing is CFR to the best of my knowedge the material does not start at pts 0, the code ignores the start time aka the first pts. Theres only one possibility to fix this and that is to use the

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-23 Thread Kieran Kunhya
why does the filter not use the input pts? Michael, none of the PTS hacks will ever work - (inverse)telecine is a CFR operation. It inherently requires knowing the duration of the frame and in VFR mode you don't know it. Kieran ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-22 Thread Michael Niedermayer
On Mon, Mar 23, 2015 at 03:43:55AM +0530, Himangi Saraogi wrote: This is an exact inverse of the telecine filter unlike previously existing pullup and fieldmatch ones. Tested on few samples generated using the telecine filter. Documentation is yet to be added. Added an additional parameter