Opened 4 years ago

Closed 4 years ago

#1056 closed defect (fixed)

Typos in JVET-R2001-v8: 8.7.4.1 and Table 51/126/131

Reported by: moonmo.koo Owned by:
Priority: minor Milestone:
Component: spec Version:
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

  1. In section 8.7.4.1 in JVET-R2001-v8, lfnst_idx[ x0 ][ y0 ] found in the following passage should be replaced by just lfnst_idx, because lfnst_idx is not an array-type syntax element.
...
The variable implicitMtsEnabled is derived as follows:
– If sps_mts_enabled_flag is equal to 1 and one or more of the following conditions are true, implicitMtsEnabled is set equal to 1:

  – IntraSubPartitionsSplitType is not equal to ISP_NO_SPLIT
  – cu_sbt_flag is equal to 1 and Max( nTbW, nTbH ) is less than or equal to 32
  – sps_explicit_mts_intra_enabled_flag is equal to 0 and CuPredMode[ 0 ][ xTbY ][ yTbY ] is equal to MODE_INTRA and lfnst_idx[ x0 ][ y0 ] is equal to 0 and intra_mip_flag[ x0 ][ y0 ] is equal to 0

– Otherwise, implicitMtsEnabled is set equal to 0.
...

Therefore, the above should be modified to the following:

...
The variable implicitMtsEnabled is derived as follows:
– If sps_mts_enabled_flag is equal to 1 and one or more of the following conditions are true, implicitMtsEnabled is set equal to 1:

  – IntraSubPartitionsSplitType is not equal to ISP_NO_SPLIT
  – cu_sbt_flag is equal to 1 and Max( nTbW, nTbH ) is less than or equal to 32
  – sps_explicit_mts_intra_enabled_flag is equal to 0 and CuPredMode[ 0 ][ xTbY ][ yTbY ] is equal to MODE_INTRA and lfnst_idx is equal to 0 and intra_mip_flag[ x0 ][ y0 ] is equal to 0

– Otherwise, implicitMtsEnabled is set equal to 0.
...
  1. Two syntax elements, intra_subpartitions_mode_flag and intra_subpartitions_split_flag, are defined in the form of array in both the associated syntax table and semantics section. Therefore, the two should be referred with a suffix ~[ ][ ] in Table 51/126/131, as other similiarly defined syntax elements like intra_luma_mpm_flag[ ][ ] or intra_luma_not_planar_flag[ ][ ].

Change history (1)

comment:1 Changed 4 years ago by bbross

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

Good catch, thanks!

This will be fixed in the first version of D10.

Note: See TracTickets for help on using tickets.