mpv-player / mpv

🎥 Command line video player
https://mpv.io
Other
28.39k stars 2.91k forks source link

Windows 10bit output & Infinite contrast ICC Profile #5980

Closed sat4all1 closed 5 years ago

sat4all1 commented 6 years ago

mpv version and platform

mpv-x86_64-20180701-git-861c102 Windows 10 x64 1803

Reproduction steps

Expected behavior

Actual behavior

Log file

https://pastebin.com/SACVbdq7

zc62 commented 6 years ago

10 bit issue, see #5237 and #5804

sat4all1 commented 6 years ago

If i understand correctly, 10bit output isn't supported yet? About the icc profile issue, i think i found the sweet spot for my display "icc-contrast=9969", but there are still problems. when using icc profile i get pinkish tint mpv icc profile on profile mpv icc profile off no profile madvr 3dlut on "The 3d-lut was created from the same profile i'm using with mpv" westworld - s02e02 - reunion web-dl mkv_snapshot_00 50_ 2018 07 17_19 09 19 here is the icc profile my lg oled.zip

log.txt

haasn commented 6 years ago

If i understand correctly, 10bit output isn't supported yet?

It's supported on Linux, but apparently not (yet?) on Windows or OS X (?).

About the icc profile issue, i think i found the sweet spot for my display "icc-contrast=9969", but there are still problems. when using icc profile i get pinkish tint

Can you upload the profile?

sat4all1 commented 6 years ago

@haasn

here my lg oled.zip

thanks

Akemi commented 6 years ago

It's supported on Linux, but apparently not (yet?) on ... OS X (?).

there is only an open PR (#5623) for 10bit support on macOS.

sat4all1 commented 6 years ago

@haasn

any hint?

sat4all1 commented 5 years ago

@haasn

Is it possible to allow infinite contrast because even with icc-contrast set to 1000000 i lose perfect blacks.

zc62 commented 5 years ago

Made a commit in my fork: https://github.com/zc62/mpv/commit/96c3f8a16cc60b75d6a370d765a2af6560c7ed04. Use --icc-contrast=inf to bypass the code that modifies the black point.

Is this what you really want? Could you test how this is compared to madVR + 3dlut (which software did you use to generate that 3dlut from the profile?), and can you really tell the difference between --icc-contrast=inf and --icc-contrast=1000000?

sat4all1 commented 5 years ago

Anywhere to download windows build based on your fork?

zc62 commented 5 years ago

I wrote and compiled the code in Linux. It is a bit tedious to make a fully static build to let others directly use the compiled binary, so I am expecting you to follow the guide (https://github.com/mpv-player/mpv/blob/master/DOCS/compile-windows.md) to compile it on your own. It is actually not that hard to compile mpv in MSYS2. Then you can directly run mpv from the MSYS2 shell.

What you need to do to compile that branch is just replacing

git clone https://github.com/mpv-player/mpv.git && cd mpv

by

git clone https://github.com/zc62/mpv.git && cd mpv && git checkout inf_contrast
sat4all1 commented 5 years ago

Now with icc-contrast=inf i have perfect blacks, and no visual difference compared to madvr + 3d-lut generated using displaycal. Hope you can push this to master branch.

Thanks much

zc62 commented 5 years ago

Well, I was actually thinking inf should not make too much difference from 1000000. Probably that's because I don't own an OLED TV. Will make it a PR.