Paul B Mahol wrote:
On 1/30/16, Andy Furniss <adf.li...@gmail.com> wrote:
Paul B Mahol wrote:
On 1/30/16, Andy Furniss <adf.li...@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.

Yes, v4 looks good to me.

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to