Opened 4 years ago

Closed 4 years ago

#990 closed defect (fixed)

Misleading definition of limited EGk binarization, leading to errors in its invocation

Reported by: jonathang Owned by:
Priority: minor Milestone:
Component: spec Version:
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…


Definition of limited EGk binarization (section currently depends on a "riceParam" variable rather than k. The variable name "log2TransformRange" is also misleading because the process is now used by both transform skip residual coding and MVD coding. Editorial suggestions are to replace "riceParam" with "k", and "log2TransformRange" with "truncSuffixLen" instead. and invoke with k set equal to cRiceParam + 1 when k is not used by the binarisation. They also invoke with cRiceParam as an input variable (it is not). does not have errors, but invokes by redundantly setting both k=1 and riceParam=1.

Attachments (1)

expGolomb.docx (19.9 KB) - added by jonathang 4 years ago.
Suggested fix (tracked changes)

Download all attachments as: .zip

Change history (2)

Changed 4 years ago by jonathang

Suggested fix (tracked changes)

comment:1 Changed 4 years ago by bbross

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

Good catch, thanks!

This will be fixed in the first version of D10.

Note: See TracTickets for help on using tickets.