Opened 5 years ago
Closed 5 years ago
#702 closed defect (fixed)
Variable name compIdx is used for different purposes in the spec
Reported by: | guichunli | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D7 vE |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
Variable name "compIdx" is used for different purposes in the spec as below:
- MV component index as in semantic for mvd_sign_flag[compIdx]
mvd_sign_flag[ compIdx ] specifies the sign of a motion vector component difference as follows: – If mvd_sign_flag[ compIdx ] is equal to 0, the corresponding motion vector component difference has a positive value. – Otherwise (mvd_sign_flag[ compIdx ] is equal to 1), the corresponding motion vector component difference has a negative value.
- Colour component index as in 8.4.5.3 Decoding process for palette mode
for( i = 0; i < CurrentPaletteSize[ startComp ]; i++ ) for( compIdx = startComp; compIdx < (startComp + numComps); compIdx++ )
It would be more clear if the "compIdx" in 8.4.5.3 is change to "cIdx", which is aligned with colour component index used at other places.
Change history (3)
comment:1 Changed 5 years ago by bbross
- Version changed from VVC D7 vD to VVC D7 vE
comment:2 Changed 5 years ago by bbross
comment:3 Changed 5 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Thanks for reporting!
This has been fixed in JVET-Q2001-v5.
Note: See TracTickets for help on using tickets.
For palette I suggest the following:
In PredictorPaletteSize[ compIdx ], with compIdx = 0, 1, compIdx specifies luma or chroma not Y,Cb or Cr as for cIdx:
compIdx -> isChroma
In PredictorPaletteEntries[ compIdx ][ i ], compIdx can range from 0 to 3 as for cIdx:
compIdx -> cIdx