Opened 4 years ago

Closed 4 years ago

#1296 closed defect (fixed)

Packed YUV output in VTM decoder doesn't work anymore

Reported by: crhelmrich Owned by:
Priority: major Milestone: VTM-10.0
Component: VTM Version:
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…


Specifying --PYUV=1 in the VTM decoder command-line used to allow packed YUV output during decoding, see also VVCSoftware_VTM/doc/pyuv_format.pdf

In commit
(Jan 22, 2020, author yiwenchen) a

CHECK( writePYUV, "Not supported" );

was introduced in source/Lib/Utilities/VideoIOYuv.cpp which completely disables support for PYUV output, even though the respective code is still present and, when removing the above CHECK, decoding to PYUV format seems to work fine (at least on the 10-bit SDR CTC bitstream I tested it with).

I'll submit a merge request soon which simply deletes the above source code line with the CHECK, but maybe the author of that code can shed some light on the reasoning behind this check, i.e., under which circumstances the PYUV output doesn't work properly.



Change history (2)

comment:1 Changed 4 years ago by crhelmrich

Sorry, it seems the original author of this CHECK wasn't yiwenchen, as noted above, but V. Seregin when integrating JVET_O1164_RPR on Aug 13, 2019:
Would it be possible to condition this CHECK on the use of RPR?

Edit: Merge request has been pushed to address this issue.


Last edited 4 years ago by crhelmrich (previous) (diff)

comment:2 Changed 4 years ago by fbossen

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.