Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deinterlace fails with hwdec for mpeg2video #14816

Closed
6 tasks done
jiangzhenjerry opened this issue Sep 8, 2024 · 7 comments
Closed
6 tasks done

deinterlace fails with hwdec for mpeg2video #14816

jiangzhenjerry opened this issue Sep 8, 2024 · 7 comments
Labels

Comments

@jiangzhenjerry
Copy link

jiangzhenjerry commented Sep 8, 2024

mpv Information

mpv v0.38.0-747-g0eb5e914 Copyright © 2000-2024 mpv/MPlayer/mplayer2 projects
 built on Sep  6 2024 12:13:04
libplacebo version: v7.349.0 (v7.349.0-9-gefb8934-dirty)
FFmpeg version: N-116911-gc0666d8be
FFmpeg library versions:
   libavcodec      61.12.100
   libavdevice     61.2.100
   libavfilter     10.2.102
   libavformat     61.5.101
   libavutil       59.36.100
   libswresample   5.2.100
   libswscale      8.2.100

Other Information

- Windows version: Windows 11 [10.0.27695.1000]
- GPU model, driver and version: Intel(R) Arc(TM) Graphics, Driver Version 32.0.101.5972
- Source of mpv: https://github.com/zhongfly/mpv-winbuild/releases
- Introduced in version: Not sure, but must be somewhere between `mpv-x86_64-v3-20240703-git-7c70df0.7z` (a working version from my backup) and present.

Reproduction Steps

Attached later in this issue is 1.mkv, which is a short mpeg2video remuxed directly from a DVD I own.

  1. Use vanilla mpv without any mpv.conf
  2. Run mpv --deinterlace 1.mkv. This correctly deinterlaces the video.
  3. Run mpv --deinterlace --hwdec --hwdec-codecs=all 1.mkv. This fails to deinterlace the video.

Expected Behavior

In Step 3 above, the video should be deinterlaced.

Actual Behavior

In Step 3 above, the video is not deinterlaced.

Log File

output.txt

Sample Files

1.mkv.zip

I carefully read all instruction and confirm that I did the following:

  • I tested with the latest mpv version to validate that the issue is not already fixed.
  • I provided all required information including system and mpv version.
  • I produced the log file with the exact same set of files, parameters, and conditions used in "Reproduction Steps", with the addition of --log-file=output.txt.
  • I produced the log file while the behaviors described in "Actual Behavior" were actively observed.
  • I attached the full, untruncated log file.
  • I attached the backtrace in the case of a crash.
@hooke007
Copy link
Contributor

hooke007 commented Sep 8, 2024

It's a cpu filter. Use hwdec=auto-copy instead.

@hooke007
Copy link
Contributor

hooke007 commented Sep 8, 2024

Or use d3d11vpp's deint with d3d11va hwdec

https://mpv.io/manual/master/#video-filters-d3d11vpp

@jiangzhenjerry
Copy link
Author

@hooke007 Thanks for the notes. These methods work, but I'd prefer the direct decoding mode and the possibility to use deinterlace=auto.

An additional finding: I grabbed mpv-x86_64-v3-20240703-git-7c70df0.7z from my recent backup (which is also from zhongfly but no longer available for download), and that version works just fine. Apparently, something must have changed from July 3 to present. I edited my bug report to reflect this finding.

@sfan5
Copy link
Member

sfan5 commented Sep 8, 2024

this should in fact work automatically:

} else if (img->imgfmt == IMGFMT_D3D11) {
char *args[] = {"parity", field_parity, NULL};
p->sub.filter =
mp_create_user_filter(f, MP_OUTPUT_CHAIN_VIDEO, "d3d11vpp", args);

@hooke007
Copy link
Contributor

hooke007 commented Sep 8, 2024

this should in fact work automatically:

67e087d safe
f44c29b broken

kasper93 added a commit to kasper93/mpv that referenced this issue Sep 8, 2024
@kasper93
Copy link
Contributor

kasper93 commented Sep 8, 2024

@hooke007 @jiangzhenjerry does #14822 work for you?

@jiangzhenjerry
Copy link
Author

@kasper93 Thank you, it works for me.

na-na-hi pushed a commit to na-na-hi/mpv that referenced this issue Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants