Opened 5 years ago
Closed 5 years ago
#324 closed defect (fixed)
Mismatch in equation 8-784 with SW
| Reported by: | chhuanb | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | spec | Version: | VVC D5 v10 |
| Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
In BDOF motion field derivation equation 8-784, sGx2 should be changed to sGy2:
spec:
vy = sGy2 > 0 ? Clip3( −mvRefineThres, mvRefineThres, ( ( sGydI << 3 )
( ( vx * sGxGym ) << 12 + vx * sGxGys ) >> 1 ) >> Floor( Log2( sGx2 ) ) ) : 0
SW:
vy = sGy2 > 0 ? Clip3( −mvRefineThres, mvRefineThres, ( ( sGydI << 3 )
( ( vx * sGxGym ) << 12 + vx * sGxGys ) >> 1 ) >> Floor( Log2( sGy2 ) ) ) : 0
In addition, is it necessary to separate sGxGy into sGxGym and sGxGys ?
(vx * sGxGym ) << 12 + vx * sGxGys ) is equal to vx * sGxGy and then the equation can be simplified as:
vy = sGy2 > 0 ? Clip3( −mvRefineThres, mvRefineThres, ( ( sGydI << 3 )
( ( vx * sGxGy ) >> 1 ) >> Floor( Log2( sGy2 ) ) ) : 0
Change history (4)
comment:1 Changed 5 years ago by bbross
- Version changed from VVC D5 v8 to VVC D5 v9
comment:2 Changed 5 years ago by bbross
- Version changed from VVC D5 v9 to VVC D5 v10
comment:3 Changed 5 years ago by bbross
comment:4 Changed 5 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Fixed in D6 vC
Fixed in D6 vC