Opened 5 years ago

Closed 5 years ago

#725 closed defect (fixed)

IntraSubPartitionsSplit[ x0 ][ y0 ] is not defined.

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

Description

Currently, the variable, IntraSubPartitionsSplit[ x0 ][ y0 ], is referred in two syntax tables, coding_unit and transform_unit, as follows:

1) In coding_unit,

 if( treeType  !=  DUAL_TREE_CHROMA  &&  lfnst_idx  = =  0  &&
     transform_skip_flag[ x0 ][ y0 ][ 0 ]  = =  0  &&  Max( cbWidth, cbHeight )  <=  32  &&
     '''IntraSubPartitionsSplit[ x0 ][ y0 ]'''  = =  ISP_NO_SPLIT  &&  cu_sbt_flag  = =  0  &&
     MtsZeroOutSigCoeffFlag  = =  1  &&  tu_cbf_luma[ x0 ][ y0 ] )  {
     if( ( ( CuPredMode[ chType ][ x0 ][ y0 ]  = =  MODE_INTER  &&
         sps_explicit_mts_inter_enabled_flag )  | |
         ( CuPredMode[ chType ][ x0 ][ y0 ]  = =  MODE_INTRA  &&
         sps_explicit_mts_intra_enabled_flag ) ) )

         mts_idx
 }

2) In transform_unit,

 if( tu_cbf_luma[ x0 ][ y0 ]  &&  treeType  !=  DUAL_TREE_CHROMA ) {
     if( sps_transform_skip_enabled_flag  &&  !BdpcmFlag[ x0 ][ y0 ][ 0 ]  &&  
	 tbWidth  <=  MaxTsSize  &&  tbHeight  <=  MaxTsSize  &&  
	 ( '''IntraSubPartitionsSplit[ x0 ][ y0 ]'''  = =  ISP_NO_SPLIT )  &&  !cu_sbt_flag )

	 transform_skip_flag[ x0 ][ y0 ][ 0 ]
	 ......	
 }

However, the definition of IntraSubPartitionsSplit[ x0 ][ y0 ] can not be found in the latest version of spec text. It seems that IntraSubPartitionsSplit[ x0 ][ y0 ] should be replaced by IntraSubPartitionsSplitType, because IntraSubPartitionsSplitType is determined on a CU basis.

Change history (1)

comment:1 Changed 5 years 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.