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
- 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. ...
- 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
Note: See TracTickets for help on using tickets.
Good catch, thanks!
This will be fixed in the first version of D10.