Skip to content

Conversation

@llyyr
Copy link
Contributor

@llyyr llyyr commented Feb 10, 2026

Currently a draft because it breaks when changing hwdec during runtime

// We check also if a filter is already present, to avoid removing it
bool filter_needed = opts->deinterlace == 1 ||
(opts->deinterlace == -1 && (p->interlaced_frame || p->sub.filter));
bool filter_needed = (info && info->deinterlace) && (opts->deinterlace == 1 ||
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this flipped? We don't want filter when deinterlace is supported. Also we might still prefer hardware deint. Might actually need an option, not sure.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made it only override the software filter, still crashes with hwdec however. I won't have time to look at this for a bit

@dancingmirrors
Copy link

Yes, for some reason bwdif_vulkan stopped working for me after around FFmpeg 7.1, just outputs a green screen instead. No idea what happened but libplacebo's deinterlacing works.

@llyyr llyyr force-pushed the pl-deint branch 3 times, most recently from a5e1c86 to 0939147 Compare February 10, 2026 20:14
Currently a draft because it breaks when changing hwdec during runtime
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants