Opened 21 months ago

Closed 4 weeks ago

Last modified 3 weeks ago

#1520 closed defect (fixed)

Some smaller errors in the multiview spec

Reported by: CFeldmann Owned by:
Priority: minor Milestone:
Component: Text Version:
Keywords: Cc: bbross, jvet@…

Description

It is not specified with how many bits the symbol sps_palette_predictor_initializer[comp][i] is coded. It is mentioned that it must be in the range 0 to (1 << BitDepthY/C) − 1 so I deduced that it should be using BitDepthY/C bits and this was confirmed by the reference software. However, this should be mentioned explicitly as for the other symbols with u(v) coding.

In F.7.4.3.1.1, the variable maxSlMinus1 has a typo. The L is sometimes uppercase sometimes lower case.

In F.7.4.3.1.1, equation F-12. There is a } missing at the end of the equation

Change History (4)

comment:1 Changed 4 weeks ago by yk

Thanks for the good catches! All fixed in JVET-AN1081-v2 (to be submitted).

comment:2 Changed 4 weeks ago by yk

Since it'd be weird to specify different length values for one particular syntax element depending on the index value, I added the following to fix the length issue:

The length of the sps_palette_predictor_initializer[ comp ][ i ] syntax element is Max( BitDepthY, BitDepthC ) bits.

Last edited 4 weeks ago by yk (previous) (diff)

comment:3 Changed 4 weeks ago by yk

  • Resolution set to fixed
  • Status changed from new to closed

comment:4 Changed 3 weeks ago by yk

After a discussion with Gary, the fix on the length has been updated to be as follows (to be similar in spirit for pps_palette_predictor_initializer[][], for which the length has actually been specified in the way depending on the index):

For values of i in the range of 0 to sps_num_palette_predictor_initializers_minus1, inclusive, the number of bits used to represent sps_palette_predictor_initializer[ 0 ][ i ] is BitDepthY, and the number of bits used to represent sps_palette_predictor_initializer[ 1 ][ i ] and sps_palette_predictor_initializer[ 2 ][ i ] is BitDepthC.

And consequently, the following constraint becomes unnecessary and is removed:

For values of i in the range of 0 to sps_num_palette_predictor_initializers_minus1, inclusive, the value of the sps_palette_predictor_initializer[ 0 ][ i ] shall be in the range of 0 to (1 << BitDepthY) − 1, inclusive, and the values of sps_palette_predictor_initializer[ 1 ][ i ] and sps_palette_predictor_initializer[ 2 ][ i ] shall be in the range of 0 to (1 << BitDepthC) − 1, inclusive.

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Subscriber)
  • Christian Feldmann(Reporter)
  • jvet@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Ye-Kui Wang(Participant)