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
Note: See TracTickets for help on using tickets.
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: