Opened 9 years ago

Closed 9 years ago

#969 closed enhancement (fixed)

Binarization of coeff_abs_level_remaining

Reported by: eugene.kuznetsov Owned by: bbross
Priority: minor Milestone:
Component: Text Version: D10 (L1003) v2
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

The description of binarization of coeff_abs_level_remaining in the specification is confusing and ambiguous.

I propose replacing the last three paragraphs of 9.2.2.8 with the following text.

"The binarization of coeff_abs_level_remaining consists of a prefix part and a suffix part.

The prefix part of the binarization is derived by invoking the TU binarization process as described in subclause 9.2.2.2 with Min( 4, coeff_abs_level_remaining[ n ] >> cRiceParam ) as input and cMax=4.

When coeff_abs_level_remaining[n] is greater or equal to cTRMax, the suffix bin string is derived using the EGk binarization as specified in subclause 9.2.2.4 for the suffix part ( coeff_abs_level_remaining[ n ] − cTRMax ) with the Exp-Golomb order k set equal to cRiceParam + 1.

Otherwise, the suffix string is cRiceParam bit long and it is specified by the binary representation of coeff_abs_level_remaining[ n ] − ( (coeff_abs_level_remaining[ n ] >> cRiceParam ) << cRiceParam )."

I've checked this logic against reference software and I believe it to be accurate.

Change History (2)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by bbross

  • Resolution set to fixed
  • Status changed from new to closed

The description of the binarization has been improved in v23.

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