Opened 5 years ago

Closed 5 years ago

#1458 closed defect (fixed)

Difference between spec and HM software in 7.3.8.5 Coding unit syntax

Reported by: deryzhov Owned by:
Priority: major Milestone:
Component: HM SCC Version: SCC-7.0 (HM16.8)
Keywords: Cc: jct-vc@…

Description

To check if palette_mode_flag should be inserted to bitstream or not HM software compared CU size with fixed value <= 32 (or < 64) while spec compared CU size with max transform size:

HEVC SCC Spec:

if( palette_mode_enabled_flag && CuPredMode[ x0 ][ y0 ] = = MODE_INTRA &&

log2CbSize <= MaxTbLog2SizeY )
palette_mode_flag[ x0 ][ y0 ]

HM code:

================================

Void TDecEntropy::decodePaletteModeInfo( TComDataCU* pcCU, UInt uiAbsPartIdx, UInt uiDepth, Bool& bCodeDQP, Bool& isChromaQpAdjCoded )
{

Note: the condition is log2CbSize < MaxTbLog2SizeY in 7.3.8.5 of JCTVC-T1005-v2
if( pcCU->getSlice()->getSPS()->getSpsScreenExtension().getUsePaletteMode() && pcCU->isIntra( uiAbsPartIdx ) && (pcCU->getSlice()->getSPS()->getMaxCUWidth() >> uiDepth) < 64 )
{

m_pcEntropyDecoderIf->parsePaletteModeFlag( pcCU, uiAbsPartIdx, uiDepth );
if ( pcCU->getPaletteModeFlag( uiAbsPartIdx ) )
{

m_pcEntropyDecoderIf->parsePaletteModeSyntax( pcCU, uiAbsPartIdx, uiDepth, 3, bCodeDQP, isChromaQpAdjCoded );
pcCU->saveLastPaletteInLcuFinal( pcCU, uiAbsPartIdx, MAX_NUM_COMPONENT );

}

}

}

According to “Note” HM developers aware of this difference

Change History (1)

comment:1 Changed 5 years ago by libin

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

fixed in r4818.

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Bin Li(Participant)
  • Dmitry(Reporter)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)