Opened 8 months ago

Closed 6 months 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:

  1. 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.
  1. 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 8 months ago by bbross

  • Version changed from VVC D7 vD to VVC D7 vE

comment:2 Changed 6 months ago by bbross

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

comment:3 Changed 6 months 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.