Opened 4 years ago

Closed 4 years ago

#1352 closed defect (fixed)

ALF processing range in 8.8.5.2 and 8.8.5.4

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D10 vE
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

Neither of the following processing ranges seems correct.

Section 8.8.5.2
"For the derivation of the filtered reconstructed luma samples alfPictureL[ x ][ y ], each reconstructed luma sample inside the current luma coding tree block recPicture[ x ][ y ] is filtered as follows with x, y = 0..CtbSizeY − 1"

Section 8.8.5.4
"For the derivation of the filtered reconstructed chroma samples alfPicture[ x ][ y ], each reconstructed chroma sample inside the current chroma coding tree block recPicture[ x ][ y ] is filtered as follows with x = 0..ctbWidthC − 1, y = 0..ctbHeightC − 1"

The locations [x][y] need to be offset by the CTB position, otherwise it's just processing the top-left CTB over and over. Something like what is used in 8.8.5.7 seems more appropriate.

Change history (1)

comment:1 Changed 4 years ago by bbross

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

Good catch, thanks! This will be fixed in JVET-S2001-vH by changing [x][y] to [ xCtb + x ][ yCtb + y ] and [ xCtbC + x ][ yCtbC + y ], respectively.

Note: See TracTickets for help on using tickets.