Opened 5 years ago

Closed 5 years ago

#347 closed defect (fixed)

Wrong parsing decision of sps_ref_wraparound_enabled_flag

Reported by: fbarbier Owned by:
Priority: minor Milestone: VTM-6.0
Component: VTM Version: VTM-5.0
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

In current VTM implementation,
source/Lib/DecoderLib/VLCReader.cpp:1710

  if( pcSPS->getCTUSize() + 2*pcSPS->getLog2MinCodingBlockSize() <= pcSPS->getPicWidthInLumaSamples() )

MinCodingBlockSize is in the wrong unit (log2).
Should be :

if( pcSPS->getCTUSize() + 2*(1 << pcSPS->getLog2MinCodingBlockSize()) <= pcSPS->getPicWidthInLumaSamples() )

Please find the related fix here:
​https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/625/

Change history (2)

comment:1 Changed 5 years ago by fbarbier

  • Version changed from VTM-5.0rc1 to VTM-5.0

comment:2 Changed 5 years ago by ksuehring

  • Milestone set to VTM-6.0
  • Resolution set to fixed
  • Status changed from new to closed

The MR was merged

Note: See TracTickets for help on using tickets.