Opened 9 years ago

Closed 9 years ago

#527 closed defect (fixed)

Typo in derivation of MinCbAddrZS

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

Description

The inner loop of "Derivation process for the minimum coding block address of a coding block" is:

for (i = 0, p = 0; i < Log2CtbSize - Log2MinCbSize; i++) {
    m = 1 << i
    p += (m & x ? m*m : 0) + (m & y ? 2*m*m : 0)
    MinCbAddrZS[x][y] += p
}

But to get a proper zscan order(e.g. (0, 0) => 0, (0, 1) => 2, (1, 0) => 1, (1, 1) => 3), either the "p +=" should be a "p =" or the "MinCbAddrZS[x][y] += p" should be after the loop.
(Additionally, since Log2CtbSize - Log2MinCbSize == log2_diff_max_min_coding_block_size, I don't see why the later isn't used).

Change History (2)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by bbross

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

This is fixed in JCTVC-I1003_d3

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)