Opened 4 years ago

Closed 4 years ago

#1308 closed defect (fixed)

Inconsistent use of IsAvailable for chroma

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

Description

In Eq 1190, for chroma the pixel availability is set using chroma location (xCurr, yCurr):

IsAvailable[ cIdx ][ xCurr + i ][ yCurr + j ] = TRUE


However, when IsAvailable is used in 6.4.4, the luma location (xNbY, yNbY) is used instead:

IsAvailable[ cIdx ][ xNbY ][ yNbY ] is equal to FALSE


I believe Eq 1190 needs to be changed to use luma locations in order for this to work correctly.

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 JVET-S2001-vH by changing the array indices and adding an additional loop for IsAvailable as follows:

The following assignments are made for i = 0..nCurrSw * SubWidthC − 1, j = 0..nCurrSh* SubHeightC − 1:

IsAvailable[ cIdx ][ xCurr * SubWidthC + i ][ yCurr * SubHeightC + j ] = TRUE (1190)

Note: See TracTickets for help on using tickets.