Opened 5 years ago

Closed 5 years ago

# ClipH function issue

Reported by: Owned by: bbross minor VVC D4 v1 spec VVC D5 v6 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.

### comment:1 Changed 5 years ago by bbross

• Milestone changed from VVC D4 to VVC D4 v1

Milestone renamed

### comment:2 Changed 5 years ago by bbross

• Version changed from VVC D3 v9 to VVC D4 v2

### comment:3 Changed 5 years ago by bbross

• Version changed from VVC D4 v2 to VVC D4 v3

### comment:4 Changed 5 years ago by bbross

• Version changed from VVC D4 v3 to VVC D4 v4

### comment:5 Changed 5 years ago by bbross

• Version changed from VVC D4 v4 to VVC D4 v6

### comment:6 Changed 5 years ago by bbross

• Version changed from VVC D4 v6 to VVC D4 v7

### comment:7 Changed 5 years ago by bbross

• Version changed from VVC D4 v7 to VVC D5 v4

### comment:8 Changed 5 years ago by bbross

• Version changed from VVC D5 v4 to VVC D5 v5

### comment:9 Changed 5 years ago by bbross

• Version changed from VVC D5 v5 to VVC D5 v6

### comment:10 Changed 5 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.