Opened 4 years ago

Closed 4 years ago

#1166 closed defect (fixed)

Palette mode semantics: Extra bracket in formula (185)

Reported by: patrice.onno Owned by:
Priority: minor Milestone: VVC D10
Component: spec Version: VVC D10 vA
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

This paragraph
if( PaletteScanPos > 0 ) {

xcPrev = x0 + TraverseScanOrder[ log2CbWidth ][ log2bHeight ][ PaletteScanPos − 1 ][ 0 ]
ycPrev = y0 + TraverseScanOrder[ log2CbWidth ][ log2bHeight ][ PaletteScanPos − 1 ][ 1 ]
if( CopyAboveIndicesFlag[ xcPrev ][ ycPrev ] = = 0 )

adjustedRefPaletteIndex = PaletteIndexMap[ xcPrev ][ ycPrev ] { (185)

else {

if( !palette_transpose_flag )

adjustedRefPaletteIndex = PaletteIndexMap[ xC ][ yC − 1 ]

else

adjustedRefPaletteIndex = PaletteIndexMap[ xC − 1 ][ yC ]

}

}

should be:
(The mistake is the extra { before the formula number (185) )

if( PaletteScanPos > 0 ) {

xcPrev = x0 + TraverseScanOrder[ log2CbWidth ][ log2bHeight ][ PaletteScanPos − 1 ][ 0 ]
ycPrev = y0 + TraverseScanOrder[ log2CbWidth ][ log2bHeight ][ PaletteScanPos − 1 ][ 1 ]
if( CopyAboveIndicesFlag[ xcPrev ][ ycPrev ] = = 0 )

adjustedRefPaletteIndex = PaletteIndexMap[ xcPrev ][ ycPrev ] (185)

else {

if( !palette_transpose_flag )

adjustedRefPaletteIndex = PaletteIndexMap[ xC ][ yC − 1 ]

else

adjustedRefPaletteIndex = PaletteIndexMap[ xC − 1 ][ yC ]

}

}

Change history (1)

comment:1 Changed 4 years ago by jlchen

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

Thanks for reporting the issue. Will be fixed in JVET-S2001-vC.

Note: See TracTickets for help on using tickets.