Opened 2 years ago

Closed 14 months ago

#8779 closed defect (fixed)

Zscale transform to ST 2084 EOTF results in black video output

Reported by: Alex Zambelli Owned by:
Priority: normal Component: avfilter
Version: git-master Keywords: zscale regression
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no


Summary of the bug:

Sometime between releases 4.2.2 and 4.2.3 a bug was introduced into Zscale filter which causes the video output of the filter to be reduced to black when transforming video using SMPTE ST 2084 (PQ) EOTF.

How to reproduce:

Source video:

ffmpeg version git-2020-06-28-4cfcfb3
built with gcc 9.3.1 (GCC) 20200621

ffmpeg.exe -y -i zscale_test_source.mp4 -vf zscale=t=linear:w=640:h=360,format=gbrpf32le,zscale=t=smpte2084:p=bt709:m=bt709,format=yuv420p10le -vcodec ffv1 zscale_test_smpte2084.avi

Sample output:

If t=smpte2084 is replaced with another EOTF, such as bt709, the filter produces the correct output:

ffmpeg.exe -y -i zscale_test_source.mp4 -vf zscale=t=linear:w=640:h=360,format=gbrpf32le,zscale=t=bt709:p=bt709:m=bt709,format=yuv420p10le -vcodec ffv1 zscale_test_bt709.avi

Sample output:

The bug does not repro in FFmpeg release 4.2.2. To the best of my memory while working with nightly builds, this Zscale transform worked correctly until at least early May 2020.

Attachments (2)

zscale_test_smpte2084.avi (214.4 KB ) - added by Alex Zambelli 2 years ago.
Broken zscale t=smpte2084 output
zscale_test_bt709.avi (1.1 MB ) - added by Alex Zambelli 2 years ago.
Correct zscale output (using t=bt709 for example)

Download all attachments as: .zip

Change History (7)

by Alex Zambelli, 2 years ago

Attachment: zscale_test_smpte2084.avi added

Broken zscale t=smpte2084 output

by Alex Zambelli, 2 years ago

Attachment: zscale_test_bt709.avi added

Correct zscale output (using t=bt709 for example)

comment:1 by Carl Eugen Hoyos, 2 years ago

Component: ffmpegundetermined
Keywords: regression added; libzimg z.lib smpte2084 st2084 hdr removed
Priority: importantnormal

Please point us to the commit introducing the regression.

in reply to:  1 comment:2 by Alex Zambelli, 2 years ago

Replying to cehoyos:

Please point us to the commit introducing the regression.

I'm not familiar with that part of the project code so it'd be difficult for me to look for specific commits.

The best I could do is find the date/build when the issue is introduced, by trying out the same command line on different builds.

comment:3 by Elon Musk, 2 years ago

Make sure this is not zimg issue, as zscale is just a wrapper around it.

comment:4 by mawazi, 23 months ago

Component: undeterminedavfilter

This appears to be a problem in zimg, fixed with the following zimg commit:

The npl parameter of zscale, which deals with peak luminance when converting between HDR and SDR, was not being dealt with properly when not explicitly set. This can also be worked around in current ffmpeg builds by setting npl=100 (which should be the default when not set) when using zscale to convert between HDR and SDR transfer functions (and vice-versa).

comment:5 by Balling, 14 months ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.