[vlc-devel] [PATCH] Fixed incorrect backward/forward reference ordering in vaapi deinterlacing
thomas at gllm.fr
Tue Jun 27 16:05:52 CEST 2017
On Tue, Jun 27, 2017, at 15:25, Victorien Le Couviour--Tuffet wrote:
> On Tue, Jun 27, 2017 at 03:58:48PM +0300, Oliver Collyer wrote:
> > After testing the Motion Adaptive deinterlacing mode and finding it produced lots of flicker/judder and not much deinterlacing I traced the error back to the below.
> > Although it seems a little counter-intuitive, the "forward_refs" are actually the frames older than the current one, and the "backward_refs" are those later.
> > In addition, the forward_refs (previously the backward_refs) have to be ordered such that  is most recent,  older, etc; before this patch it was the opposite.
> > Just to be 100% sure, I have also cross-referenced this approach with how FFmpeg does it:
> > https://github.com/FFmpeg/FFmpeg/blob/master/libavfilter/vf_deinterlace_vaapi.c <https://github.com/FFmpeg/FFmpeg/blob/master/libavfilter/vf_deinterlace_vaapi.c>
> > See function deint_vaapi_filter_frame.
Indeed, patch applied, thanks a lot.
> Oh thanks I would have never guessed that. It is indeed a bit
> counter-intuitive. And obviously the VA documentation does not tell you
> I was probably not able to see those flicker/judder by myself as on my
> I only have one backward frame. Anyway, thanks again.
> Also, LGTM.
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel