Opened 21 months ago

Closed 2 months ago

#1522 closed defect (fixed)

Offset issue in clause 8.5.4.3

Reported by: crosewarne Owned by:
Priority: minor Milestone:
Component: Text Version: v5 (02/2018)
Keywords: Cc: bbross, jvet@…

Description

(filed on behalf of Min Woo)

In section 8.5.4.3, Equation (8-279) appears to have an issue.

resSamples is a block level array but picture-level offsets (xCb, yCb) are applied when addressing samples in this array. The equation should instead be:

  1. The (nCbS)x(nCbS) residual sample array of the current coding block resSamples is modified as follows, for i = 0..nTbS − 1, j = 0..nTbS − 1:

resSamples[ xB0 / SubWidthC + i, yBN / SubHeightC + j ] = transformBlock[ i, j ] (8 279)

Change History (2)

comment:1 Changed 2 months ago by yk

Thanks for the good catch! Fixed in JVET-AN1018-v2 (to be submitted). Now the equation is as follows (and some other places of A[ x, y ] were also changed to A[ x ][ y ]):

resSamples[ xB0 / SubWidthC + i ][ yBN / SubHeightC + j ] = transformBlock[ i ][ j ]

Last edited 2 months ago by yk (previous) (diff)

comment:2 Changed 2 months ago by yk

  • 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(Subscriber)
  • Chris Rosewarne(Reporter)
  • jvet@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Ye-Kui Wang(Participant)