id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
847	Typo of signaling condition for chroma intra prediction mode (cclm_mode_{flag,idx}, intra_chroma_pred_mode) in WD	tsukuba.takeshi		"In SW(2b6a7e1a), intra_chroma_pred_mode or cclm_mode_{flag,idx} is signaled depending on value of intra_bdpcm_chroma_flag. 

{{{
void CABACReader::cu_pred_data( CodingUnit &cu )
{
  if( CU::isIntra( cu ) )
  {
...#if JVET_Q0110_Q0785_CHROMA_BDPCM_420
    if( !cu.Y().valid() || ( !cu.isSepTree() && cu.Y().valid() ) )
    {
      bdpcm_mode(cu, ComponentID(CHANNEL_TYPE_CHROMA));
    } 
#endif
    intra_chroma_pred_modes( cu );
    return;
  }
...
}

void CABACReader::intra_chroma_pred_modes( CodingUnit& cu )
{
  if( cu.chromaFormat == CHROMA_400 || ( cu.isSepTree() && cu.chType == CHANNEL_TYPE_LUMA ) )
  {
    return;
  }

#if JVET_Q0110_Q0785_CHROMA_BDPCM_420
  if( cu.bdpcmModeChroma )
  {
    cu.firstPU->intraDir[1] = cu.bdpcmModeChroma == 2 ? VER_IDX : HOR_IDX;
    return;
  }
#endif
  PredictionUnit *pu = cu.firstPU;

  {
    CHECK( pu->cu != &cu, ""Inkonsistent PU-CU mapping"" );
    intra_chroma_pred_mode( *pu );
  }
}
}}}


In WD, those syntax cannot be signaled regardless of value of intra_bdpcm_chroma_flag as below:

{{{
if ( cbWidth / SubWidthC  <=  MaxTsSize  &&  cbHeight / SubHeightC  <=  MaxTsSize
	&&  sps_bdpcm_enabled_flag ) {	
	intra_bdpcm_chroma_flag				ae(v)
	if( intra_bdpcm_chroma_flag )	
		intra_bdpcm_chroma_dir_flag		ae(v)
} else {	
	if( CclmEnabled )	
		cclm_mode_flag				ae(v)
	if( cclm_mode_flag )	
		cclm_mode_idx				ae(v)
	else	
		intra_chroma_pred_mode			ae(v)
}	
}}}

Suggested fix is to align WD to SW:
{{{
if ( cbWidth / SubWidthC  <=  MaxTsSize  &&  cbHeight / SubHeightC  <=  MaxTsSize
	&&  sps_bdpcm_enabled_flag ) 	
	intra_bdpcm_chroma_flag				ae(v)
if( intra_bdpcm_chroma_flag )	
	intra_bdpcm_chroma_dir_flag			ae(v)
else {	
	if( CclmEnabled )	
		cclm_mode_flag				ae(v)
	if( cclm_mode_flag )	
		cclm_mode_idx				ae(v)
	else	
		intra_chroma_pred_mode			ae(v)
}	
}}}
"	defect	closed	minor		spec	VVC D8 vB	fixed		ksuehring bbross XiangLi fbossen jvet@…