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

Good catches. Thanks! Will be fixed in JVET-S2001-vH.

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.

Note: See TracTickets for help on using tickets.