Opened 9 years ago

Closed 9 years ago

#632 closed defect (fixed)

derivation of significant_coeff_group_flag ctxIdxInc does not handle non-square CG

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

Description

The value of ctxCG in Section 9.3.3.1.3 is incorrect for 8x2 and 2x8 coefficient groups, to be correct it should be equivalent to:

if (log2TrafoWidth == 3 && log2TrafoHeight == 3 && scanIdx != 0) {
    if (scanIdx == 1) {
        if (yCG < 3)
            ctxCG += significant_coeff_group_flag[xCG][yCG + 1];
    } else { //scanIdx == 2
        if (xCG < 3)
            ctxCG += significant_coeff_group_flag[xCG + 1][yCG];
    }
} else {
    if (xCG < (1 << (log2TrafoWidth - 2)) - 1)
        ctxCG += significant_coeff_group_flag[xCG + 1][yCG];
    if (yCG < (1 << (log2TrafoHeight - 2)) - 1)
        ctxCG += significant_coeff_group_flag[xCG][yCG + 1];
}

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by bbross

  • Milestone set to D8

This issue should be resolved by the adopted removal of 8x2 and 2x8 CGs (JCTVC-J0256).

comment:3 Changed 9 years ago by bbross

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