Opened 5 years ago
Closed 5 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 5 years ago by bbross
- Milestone changed from VVC D4 to VVC D4 v1
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.
Milestone renamed