Opened 9 years ago

Closed 9 years ago

#850 closed defect (fixed)

Mismatch between WD and HM in equation (7-67)

Reported by: sue.naing Owned by: bbross
Priority: minor Milestone:
Component: Text Version: D9 (K1003) v9
Keywords: delta_chroma_offset_l0 Cc: bbross, wjhan, jct-vc@…

Description

In K1003-v9:
Session 7.4.8.4 (equation 7-67):
ChromaOffsetL0[ i ][ j ] = Clip3( −128, 127, ( delta_chroma_offset_l0[ i ][ j ] − ( ( 128 * ChromaWeightL0[ i ][ j ] ) >> ChromaLog2WeightDenom ) − 128 ) )

In HM9.0.1:
TDecCavlc::xParsePredWeightTable():
ChromaOffsetL0[ i ][ j ] = Clip3( −128, 127, ( delta_chroma_offset_l0[ i ][ j ] − ( ( 128 * ChromaWeightL0[ i ][ j ] ) >> ChromaLog2WeightDenom ) + 128 ) )

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by Tanizawa

You're right. HM's equation is correct.
Therefore, the equation in (7-67) of WD should be modified.
And to fix a mismatch between WD and HM in equation (7-67), the modification of Ticket#700 should be considered.

comment:3 Changed 9 years ago by bbross

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Owner, Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Sue Naing(Reporter)
  • Woo-Jin Han(Subscriber)