Opened 6 years ago

Closed 5 years ago

#78 closed defect (wontfix)

encoder rootCbf calculation bug

Reported by: zhaoyin Owned by:
Priority: minor Milestone:
Component: BMS Version: BMS-2.0.1
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

The rootCbf is calculated by the following code, i.e., using the cbf of the first TU inside CU to derive the root cbf. It may be wrong when a CU is tiled into multiple TUs (i.e., the first TU has cbf = 0 but any other TU has cbf = 1). A fix (see "ROOT_CBF_FIX") is to use the cbf of trDepth = 0 instead.
However, this fix does not bring gain on average.

const int numValidTBlocks = ::getNumberValidTBlocks( *cs.pcv );
for (uint32_t i = 0; i < numValidTBlocks; i++)
{

#if ROOT_CBF_FIX <- fix here

cu.rootCbf |= TU::getCbfAtDepth( firstTU, ComponentID( i ), 0 );

#else

cu.rootCbf |= TU::getCbf( firstTU, ComponentID( i ) );

#endif

}

Change history (2)

comment:1 Changed 6 years ago by ksuehring

  • Component changed from VTM to BMS
  • Milestone BMS-2.1 deleted

comment:2 Changed 5 years ago by ksuehring

  • Resolution set to wontfix
  • Status changed from new to closed

BMS is not maintained anymore. VTM is the currently supported reference software of VVC. If the report is still valid, please re-open and assign to VTM.

Note: See TracTickets for help on using tickets.