Opened 4 years ago
Closed 4 years ago
#1367 closed defect (fixed)
Indexing of BdpcmFlag for chroma
Reported by: | bheng | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D10 vG |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
Section 8.4.5.2.6 includes the following condition:
BdpcmFlag[ xTbCmp ][ yTbCmp ][ cIdx ] is equal to 0;
For chroma, the index [xTbCmp][yTbCmp] used here is a chroma component position. However, everywhere else in the text BdpcmFlag is stored and referenced using a luma position.
Change history (2)
comment:1 Changed 4 years ago by bbross
comment:2 Changed 4 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Fixed as discussed/suggested in JVET-T0110-v2.
Note: See TracTickets for help on using tickets.
Good catch, thanks! I suggest to fix it as follows:
BdpcmFlag[ xTbCmp * ( cIdx > 0 ? SubWidthC : 1 ) ][ yTbCmp * ( cIdx > 0 ? SubHeightC : 1 ) ][ cIdx ] is equal to 0;