Opened 12 years ago

Closed 12 years ago

#372 closed defect (fixed)

Derivation of coefficient group scan is incorrect

Reported by: nnguyen Owned by: bbross
Priority: minor Milestone: D7
Component: Text Version: D7 (I1003) d1
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In the residual coding syntax, the derivation of xCG and yCG for 32x32 TUs depend on the scan array for 8x8 TUs. Since the scan order for 8x8 TUs was changed from diagonal to sub-block, xCG and yCG are now incorrect. The simplest fix would be to derive xCG as follows:
xCG = ScanOrder[ log2TrafoWidth ][ log2TrafoHeight ][ scanIdx ][ 16 * i ][ 0 ] >> 2

Attachments (1)

372_fix.docx​ (125.1 KB) - added by nnguyen 12 years ago.
Proposed fix

Download all attachments as: .zip

Change History (4)

comment:1 Changed 12 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

Changed 12 years ago by nnguyen

Proposed fix

comment:2 Changed 12 years ago by bbross

  • Milestone set to D7
  • Version changed from D6 (H1003) dI/dJ/dK to D7 (I1003) d1

Talking about ScanOrder:

There are two very similar named processes defined in current draft:

"6.5 Scanning array initialization process"
"6.6 Scanning order array initialization process"

and the reference to invoke the scan order init process in "residual coding semantics" and "6.6 Scanning order array initialization process" sections is wrong.

Solution:

  • delete "6.5 Scanning array initialization process"
  • change 6.5.1 and 6.5.2 to 6.5 and 6.6
  • change 6.5.3 and 6.5.4 to 6.7.1 and 6.72
  • fix the references to these subclauses

comment:3 Changed 12 years ago by bbross

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

Fixed in D7 (I1003) d2

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