Opened 4 years ago

Closed 4 years ago

#1139 closed defect (fixed)

Undefined variables in 8.4.5.2.6

Reported by: jason Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D10 vE
Keywords: 8.4.5.2.6 WAIP mode mapping process Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

In 8.4.5.2.6, the variables nCbW (coding block width) and nCbH (coding block height) are used, but undefined.
nCbW and nCbH should be inputs to this process.

Change history (4)

comment:1 Changed 4 years ago by bbross

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

Good catch thanks! nCbW and nCbH should be inputs, however this affects a lot of recursive function calls.

Fixed now in JVET-S2001-v7.

comment:2 Changed 4 years ago by jason

One place has not been fixed in D10-v7.

Line after eq. (1137):
– Otherwise (transform_skip_flag[ xTbY ][ yTbY ][ codedCIdx ] is equal to 0), the transformation process for scaled transform coefficients as specified in clause 8.7.4.1 is invoked with the transform block location ( xTbY, yTbY ), the coding block width nCbW and the coding block height nCbH, the transform block width nTbW and the transform block height nTbH, the colour component variable cIdx being set equal to codedCIdx and the (nTbW)x(nTbH) array of scaled transform coefficients d as inputs, and the output is an (nTbW)x(nTbH) array of residual samples res.

comment:3 Changed 4 years ago by jonathang

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Version changed from VVC D9 vB to VVC D10 vE

As far as I can tell, the above comment is still valid for S2001-vE.
However the text referred to is in section 8.7.2, at the line following eq (1109).

The current text is invoking process 8.7.4.1 and the variables nCbW and nCbH are missing.

comment:4 Changed 4 years ago by jlchen

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

Thanks for reporting the issue. Will be fixed in JVET-S2001 vF release.

Note: See TracTickets for help on using tickets.