id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
1630	Missing equations for applying AmvrShift	peterderivaz		"In the semantics for amvr_precision_idx, some motion vectors are adjusted as follows:

{{{
Otherwise (inter_affine_flag[ x0 ][ y0 ] is equal to 1), the variables
MvdCpL0[ x0 ][ y0 ][ 0 ][ 0 ], 
MvdCpL0[ x0 ][ y0 ][ 0 ][ 1 ], 
MvdCpL0[ x0 ][ y0 ][ 1 ][ 0 ], 
MvdCpL0[ x0 ][ y0 ][ 1 ][ 1 ], 
MvdCpL0[ x0 ][ y0 ][ 2 ][ 0 ] 
and MvdCpL0[ x0 ][ y0 ][ 2 ][ 1 ] are modified as follows:
MvdCpL0[ x0 ][ y0 ][ 0 ][ 0 ] = MvdCpL0[ x0 ][ y0 ][ 0 ][ 0 ] << AmvrShift (165)
MvdCpL1[ x0 ][ y0 ] [ 0 ][ 1 ] = MvdCpL1[ x0 ][ y0 ][ 0 ][ 1 ] << AmvrShift (166)
MvdCpL0[ x0 ][ y0 ][ 1 ][ 0 ] = MvdCpL0[ x0 ][ y0 ][ 1 ][ 0 ] << AmvrShift (167)
MvdCpL1[ x0 ][ y0 ] [ 1 ][ 1 ] = MvdCpL1[ x0 ][ y0 ][ 1 ][ 1 ] << AmvrShift (168)
MvdCpL0[ x0 ][ y0 ][ 2 ][ 0 ] = MvdCpL0[ x0 ][ y0 ][ 2 ][ 0 ] << AmvrShift (169)
MvdCpL1[ x0 ][ y0 ] [ 2 ][ 1 ] = MvdCpL1[ x0 ][ y0 ][ 2 ][ 1 ] << AmvrShift (170)
}}}

Note that the description says that 6 locations in the array MvdCpL0 are modified, while the equations modify 3 locations in the array MvdCpL0 and 3 locations in the array MvdCpL1.

I wonder if both the description and the equations are incomplete, and the correct thing to do is to modify the 6 locations in the array MvdCpL0 and also the same 6 locations in the array MvdCpL1?

"	defect	closed	minor	H.266/VVC v4	spec	H.266/VVC v3	fixed		ksuehring bbross XiangLi fbossen jvet@…