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

[Paul B Mahol]

On 1/30/16, Andy Furniss <adf.lists at gmail.com> wrote:
> Paul B Mahol wrote:
>> On 1/30/16, Andy Furniss <adf.lists at gmail.com> wrote:
>>> Paul B Mahol wrote:
>>>> Hi,
>>>>
>>>> 3rd version attached!
>>>
>>> It seems that now bff/tff flags on input are correctly followed
>>> and output is good, but it can't be overridden by field=bf or
>>> field=tf both give correct output.
>>>
>>> For raw video though, they are honored and are still the wrong way
>>> round.
>>
>> Could you test attached patch?
>>
>> Apply on top of last one.
>
> This seems good - tested af bf and tf all work as expected.
>
> You're going to hate me though, because I've noticed something else.

Better any fedback, then none.

>
> It seems to affect all versions of the patch with fields - only fields
> affected, framerate output is the same length as input.
>
> There is a frame missing somewhere (haven't had time to look if it's
> first or last).

Last one.

>
> Testing with y4m input and output I noticed by chance that yadif=1 is a
> frame larger (file size).
>
> with ffprobe it seems yadif=1 is correct in the sense that -
>
> ffprobe -show_frames in.y4m | grep "\[FRAME" | wc -l
>
> 339
>
> ffprobe -show_frames out-yadif1.y4m | grep "\[FRAME" | wc -l
>
> 678
>
> ffprobe -show_frames out-nnedi.y4m | grep "\[FRAME" | wc -l
>
> 677
>
> The "frame= xxx" command line output also shows the difference.

Should be fixed in 4th version attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avfilter-add-nnedi-filter.patch
Type: text/x-patch
Size: 46788 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160130/7c4de2b2/attachment.bin>