Opened 4 years ago
Closed 4 years ago
#1313 closed defect (fixed)
Typos in 8.5.2.14
Reported by: | bheng | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D10 vE |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
There are too many left parenthesis in Equations 601 and 602
mvX[ 0 ] = Sign( mvX[ 0 ] ) * ( ( ( Abs( mvX[ 0 ] + offset ) >> rightShift ) << leftShift ) (601)
mvX[ 1 ] = Sign( mvX[ 1 ] ) * ( ( ( Abs( mvX[ 1 ] + offset ) >> rightShift ) << leftShift ) (602)
Change history (3)
comment:1 Changed 4 years ago by yk
- Resolution set to fixed
- Status changed from new to closed
comment:2 Changed 4 years ago by bheng
- Resolution fixed deleted
- Status changed from closed to reopened
I think the following may be the correct fix instead:
mvX[ 0 ] = Sign( mvX[ 0 ] ) * ( ( ( Abs( mvX[ 0 ] ) + offset ) >> rightShift ) << leftShift ) (601)
mvX[ 1 ] = Sign( mvX[ 1 ] ) * ( ( ( Abs( mvX[ 1 ] ) + offset ) >> rightShift ) << leftShift ) (602)
comment:3 Changed 4 years ago by bbross
- Resolution set to fixed
- Status changed from reopened to closed
Thanks for checking and reopening! WIll be fixed according to your suggestion in JVET-S2001-vH.
Good catches. Thanks! Will be fixed in JVET-S2001-vH.