Opened 2 years ago

Closed 2 years ago

#248 closed defect (fixed)

Small issues and typos

Reported by: lizhang Owned by:
Priority: minor Milestone: VVC D5 v3
Component: spec Version: VVC D4 v7
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

1) In sub-clause 8.5.5.4, the 'N' in the variable below shall be removed:
– predFlagL1A1N is equal to 1

2) redundant check of block size in sub-clause 7.3.6.10, wherein the conditions before coding tu_mts_idx (i.e., && ( tbWidth <= 32 ) && ( tbHeight <= 32 )) could be removed:

if( tu_cbf_luma[ x0 ][ y0 ] && treeType != DUAL_TREE_CHROMA

&& ( tbWidth <= 32 ) && ( tbHeight <= 32 )
&& ( IntraSubPartitionsSplit[ x0 ][ y0 ] = = ISP_NO_SPLIT ) && ( !cu_sbt_flag ) ) {

if( transform_skip_enabled_flag && tbWidth <= MaxTsSize && tbHeight <= MaxTsSize )

transform_skip_flag[ x0 ][ y0 ]

if( (( CuPredMode[ x0 ][ y0 ] != MODE_INTRA && sps_explicit_mts_inter_enabled_flag )

| | ( CuPredMode[ x0 ][ y0 ] = = MODE_INTRA && sps_explicit_mts_intra_enabled_flag ))
&& ( tbWidth <= 32 ) && ( tbHeight <= 32 ) && ( !transform_skip_flag[ x0 ][ y0 ] ) )
tu_mts_idx[ x0 ][ y0 ]

}

3) Three typos in sub-clause 8.3.5 on derivation of SMVD reference indices:
NumRefIdxActive[ 0 ] should be replaced by 'NumRefIdxActive[ 0 ] - 1';
NumRefIdxActive[ 1 ] should be replaced by 'NumRefIdxActive[ 1 ] - 1';
the first RefIdxSymL1 in 'When RefIdxSymL1 is equal to −1 or RefIdxSymL1 is equal to −1' should be replaced by RefIdxSymL0.

Change history (2)

comment:1 Changed 2 years ago by bbross

  • Milestone set to VVC D5 v3

comment:2 Changed 2 years ago by bbross

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

Fixed in D5 v3

Note: See TracTickets for help on using tickets.