id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
720	Issues in LFNST index parsing	bheng		"'''1.) LFNST with MIP condition'''

This treeType condition related to MIP seems incorrect:

   ( treeType  !=  DUAL_TREE_CHROMA  | |  !intra_mip_flag[ x0 ][ y0 ]  | |
     Min( lfnstWidth, lfnstHeight )  >=  16 )  

treeType != DUAL_TREE_CHROMA is TRUE for any tree that includes luma.  Therefore anything with luma will ignore the following OR MIP conditions in that statement.  And MIP only ever occurs with luma.

I assume it was meant to be treeType == DUAL_TREE_CHROMA. 

The related condition in the VTM code appears to be correct already.


'''2.) LFNST with transform skip'''

I don't believe it was intended to allow combinations of LFNST with transform skip. The existing conditions include this check for luma transform skip ([ 0 ] component) to prevent this combination.

  transform_skip_flag[ x0 ][ y0 ][ 0 ]  = =  0

However, JVET-P0058 added transform skip for chroma.  So, the condition above should probably be extended to include Cb and Cr checks as well ([ 1 ] and [ 2 ] components).

For this issue, VTM code appears to match the text, so if the text is updated, then the software should be as well.
"	defect	closed	minor		spec	VVC D7 vE	fixed		ksuehring bbross XiangLi fbossen jvet@…