Opened 5 years ago

Closed 4 years ago

#997 closed defect (fixed)

Calling of arguments in clause 8.5.2.14

Reported by: chujoh Owned by:
Priority: trivial Milestone:
Component: spec Version: VVC D8 vB
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

There are two mistakes of calling for arguments in clause 8.5.2.14, which specifies the rounding process for motion vectors.

8.5.2.4 Derivation process for pairwise average merging candidate

– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX[ 0 ] set equal to mvLXp0Cand[ 0 ] + mvLXp1Cand[ 0 ], mvX[ 1 ] set equal to mvLXp0Cand[ 1 ] + mvLXp1Cand[ 1 ], rightShift set equal to1, and leftShift set equal to 0 as inputs and the rounded mvLXavgCand[ 0 ] as output.

The above sentence should be improved by the following.

– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX[ 0 ] set equal to mvLXp0Cand[ 0 ] + mvLXp1Cand[ 0 ], mvX[ 1 ] set equal to mvLXp0Cand[ 1 ] + mvLXp1Cand[ 1 ], rightShift set equal to1, and leftShift set equal to 0 as inputs and the rounded mvLXavgCand as output.

8.5.5.9 Derivation process for motion vector arrays from affine control point motion vectors
For i = 0..1, the following applies:
– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX set equal to diffMvLX[ x ][ y ][ i ], rightShift set equal to 8, and leftShift set equal to 0 as inputs and the rounded diffMvLX[ x ][ y ][ i ] as output.
– The value of diffMvLX[ x ][ y ][ i ] is clipped as follows:
diffMvLX[ x ][ y ][ i ] = Clip3( −dmvLimit + 1, dmvLimit − 1, diffMvLX[ x ][ y ][ i ]) (918)

The above paragraph should be improved by the following.

– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX set equal to diffMvLX[ x ][ y ], rightShift set equal to 8, and leftShift set equal to 0 as inputs and the rounded diffMvLX[ x ][ y ] as output.
For i = 0..1, the following applies:
– The value of diffMvLX[ x ][ y ][ i ] is clipped as follows:
diffMvLX[ x ][ y ][ i ] = Clip3( −dmvLimit + 1, dmvLimit − 1, diffMvLX[ x ][ y ][ i ]) (918)

Change history (1)

comment:1 Changed 4 years ago by bbross

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

Good catch, thanks!

This will be fixed in first verison of D10.

Note: See TracTickets for help on using tickets.