Opened 9 years ago

Closed 9 years ago

#329 closed defect (fixed)

(Actual Version: H1003_dK): Typos and wrong constraint in max_cu_qp_delta_depth

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


Page 66: max_cu_qp_delta_depth.

First sentence: should be coding units, not coding unit.

Second sentence: Its should be It's (picky)

I believe the last sentence should be removed, as it restates the requirement in the second sentence, but it does so much less precisely (uses should rather than shall, and doesn't give an exact definition).

Change History (4)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by ksuehring

  • Summary changed from (Actual Version: H1003_dK): Typos in max_cu_qp_delta_depth to (Actual Version: H1003_dK): Typos and wrong constraint in max_cu_qp_delta_depth

I guess (being more picky), it should be "It is" instead of "It's"

As reported by Peter Chuang on the email list, the constraint is wrong. The first two sentences should be:

max_cu_qp_delta_depth indicates the control granularity for QPY values for coding units within a picture. Its It is a requirement of bitstream conformance that the value of max_cu_qp_delta_depth shall not be equal to or greater than slice_granularity + 1.

(bold italic for additions)

The last sentence can then be removed.

Last edited 9 years ago by ksuehring (previous) (diff)

comment:3 Changed 9 years ago by bbross

  • Milestone set to D7

In d7 HLS text it is now called diff_cu_qp_delta_depth and the constraint is as follows:

The value of diff_cu_qp_delta_depth shall be in the range of 0 to log2_diff_max_min_coding_block_size − slice_granularity + 1, inclusive.

Thus, this issue should be fixed in d7 HLS text.

comment:4 Changed 9 years ago by bbross

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

Fixed in JCTVC-I1003 d4.

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(Participant, Always)
  • Tim Hellman(Reporter)
  • Woo-Jin Han(Subscriber)