Opened 12 years ago

Closed 12 years ago

#458 closed defect (fixed)

log2TrafoWidth1 and log2TrafoHeight1 calculation for NSQT

Reported by: bbross Owned by: bbross
Priority: minor Milestone:
Component: Text Version: D6 (H1003) dI/dJ/dK
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In
"8.5.3.1. Decoding process for luma residual blocks"
"8.5.3.2. Decoding process for chroma residual blocks"
"8.7.1.1. Derivation process of transform unit boundary"

Remove the first parentheses in:

"The variable log2TrafoWidth1 is set equal to (log2TrafoWidth − 2) * InterTUSplitDirection.
The variable log2TrafoHeight1 is set equal to (log2TrafoHeight − 2) * (1 − InterTUSplitDirection)"

to fix the calculation of log2TrafoWidth1 and log2TrafoHeight1

"The variable log2TrafoWidth1 is set equal to log2TrafoWidth − 2 * InterTUSplitDirection.
The variable log2TrafoHeight1 is set equal to log2TrafoHeight − 2 * (1 − InterTUSplitDirection)"

Furthermore, the following comment in dI:
"[Ed. (KS): Need to be updated on second depth for non square transform tree. In 16x16 CU, second TU depth should have four square transforms in a row or column. ]"
can be removed in:

  • 7.3.8 Transform tree syntax
  • 8.5.3.1 Decoding process for luma residual blocks
  • 8.5.3.2 Decoding process for chroma residual blocks

Change History (3)

comment:1 Changed 12 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 12 years ago by bbross

Fix has been incorporated in JCTVC-I0030 (v1).

comment:3 Changed 12 years ago by bbross

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

Fixed in JCTVC-I1003_d0 (v1).

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