Opened 6 years ago

Closed 6 years ago

#148 closed defect (fixed)

ClipH function issue

Reported by: bbross Owned by:
Priority: minor Milestone: VVC D4 v1
Component: spec Version: VVC D5 v6
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

There is a problem in the ClipH function (5-9) in the case x < 0.
Since the modulo operator is only defined for non-negative numerator (as per definition in Sec. 5.2), the operation "x % o" is undefined when x < 0.
This is why we had a more complicated way of expressing this in our original proposal (L0231).
There are two solutions:
1) Extend the definition of the % operator for negative numerator
2) Use the original formula in L0231, i.e., ( o - ( ( -x ) % o ) ) % o
Thanks Philippe for spotting this.

Change history (10)

comment:1 Changed 6 years ago by bbross

  • Milestone changed from VVC D4 to VVC D4 v1

Milestone renamed

comment:2 Changed 6 years ago by bbross

  • Version changed from VVC D3 v9 to VVC D4 v2

comment:3 Changed 6 years ago by bbross

  • Version changed from VVC D4 v2 to VVC D4 v3

comment:4 Changed 6 years ago by bbross

  • Version changed from VVC D4 v3 to VVC D4 v4

comment:5 Changed 6 years ago by bbross

  • Version changed from VVC D4 v4 to VVC D4 v6

comment:6 Changed 6 years ago by bbross

  • Version changed from VVC D4 v6 to VVC D4 v7

comment:7 Changed 6 years ago by bbross

  • Version changed from VVC D4 v7 to VVC D5 v4

comment:8 Changed 6 years ago by bbross

  • Version changed from VVC D5 v4 to VVC D5 v5

comment:9 Changed 6 years ago by bbross

  • Version changed from VVC D5 v5 to VVC D5 v6

comment:10 Changed 6 years ago by bbross

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

Fixed in D5 v1

Note: See TracTickets for help on using tickets.