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.