[FFmpeg-devel] [PATCH] cuvid: add drop_second_field as input option and enable it by default

Miroslav Slugeň thunder.m at email.cz
Sun Feb 12 21:35:08 EET 2017


Thx for comment,

cuvid can't output 50fps deinterlaced output, both frames are same, and 
default behavior for yadif is to downcovert to half frame rate, so i 
think is very clever to do the same here.

Also if we return in decoder that input is framerate/2 when 
deinterlacing we should stay with that!

if (ctx->deint_mode != cudaVideoDeinterlaceMode_Weave)
     avctx->framerate = av_mul_q(avctx->framerate, (AVRational){2, 1});

Miroslav Slugeň
+420 724 825 885

Dne 12.2.2017 v 20:25 Timo Rothenpieler napsal(a):
> It does make sense, the deinterlacers convert 50 interlaced fields to 50
> progressive frames, like for example yadif can do as well.
>
> And disabling that functionality by default seems strange to me, as it's
> clearly the superior mode of operation.
>
> On 2/12/2017 8:02 PM, Miroslav Slugeň wrote:
>> There is no need to copy second field if deinterlacing, it doesn't make
>> sense to use two times same image.
>>
>>
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel



More information about the ffmpeg-devel mailing list