Opened 9 years ago

Closed 9 years ago

#636 closed defect (fixed)

cRiceParam mismatch between HM7.1 and Draft I1003-d9

Reported by: hreddy Owned by: bbross
Priority: minor Milestone: HM-7.1
Component: Text Version: D7 (I1003) d9
Keywords: cRiceParam Cc: bbross, wjhan, jct-vc@…

Description

Hi,

Issue 1:

cRiceParam is updated as follows in the HM7.1 model (which matches with I0124_r2)

#if SIMPLE_PARAM_UPDATE

if(uiLevel > 3*(1<<uiGoRiceParam))
{

uiGoRiceParam = min<UInt>(uiGoRiceParam+ 1, 4);

}

#else

In the current Draft text, cRiceParam is specified as follows (>= should be replaced by > in the following equation)

cRiceParam = Min( cLastRiceParam + ( cLastAbsLevel >= ( 3 * ( 1 << cLastRiceParam ) ) ) ? 1 : 0, 4 ) -- (9 10)

Issue 2 (text wording):
In the same section (9.3.2.8), cLastSE and cLastAbsLevel are used interchangeably. We should either use cLastAbsLevel or cLastSE consistently throughout the section.

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 hreddy

  • Summary changed from cRiceParam mismatach between HM7.1 and Draft I1003-d9 to cRiceParam mismatch between HM7.1 and Draft I1003-d9

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)
  • Harikrishna Reddy(Reporter, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Woo-Jin Han(Subscriber)