Opened 9 years ago

Closed 9 years ago

#726 closed defect (fixed)

Typo in definition of cu_qp_delta_enabled_flag

Reported by: shevach Owned by: bbross
Priority: minor Milestone:
Component: Text Version: D8 (J1003) d7
Keywords: QP Cc: bbross, wjhan, jct-vc@…

Description

According to the definition of cu_qp_delta_enabled_flag, one can derive that diff_cu_qp_delta_depth syntax element is present is always present regardless the value of regardless the value of cu_qp_delta_enabled_flag.

My suggsetion to the definition of cu_qp_delta_enabled_flag:

cu_qp_delta_enabled_flag equal to 1 specifies that the diff_cu_qp_delta_depth syntax element is present in the picture parameter set and cu_qp_delta is signaled.
If cu_qp_delta_enabled_flag equals to 0 specifies then the diff_cu_qp_delta_depth syntax element is not present in the picture parameter set and cu_qp_delta is not transmitted.

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

  1. First, there is only one minor issue here, a "not" is missing before present when cu_qp_delta_enabled_flag is equal to 0:

"cu_qp_delta_enabled_flag equal to 0 specifies that the diff_cu_qp_delta_depth syntax element is not present in the picture parameter set"

  1. Second, the cu_qp_delta_enabled_flag semantics can be improved but the syntax element cu_qp_delta mentioned in the suggestion does not exist in the current draft. Furthermore, the parsing of the related syntax elements cu_qp_delta_abs and cu_qp_delta_sign is not only conditioned on cu_qp_delta_enabled_flag but also on IsCuQpDeltaCoded which is set equal to 0 (that makes the condition true) based on cu_qp_delta_enabled_flag and Log2MinCuQpDeltaSize. Log2MinCuQpDeltaSize again, is derived based on diff_cu_qp_delta_depth. Consequently, cu_qp_delta_enabled_flag equal to 1 specifies that cu_qp_delta_abs can be present in transform unit syntax. So my suggestion to improve the semantics would be:

"cu_qp_delta_enabled_flag equal to 1 specifies that the diff_cu_qp_delta_depth syntax element is present in the picture parameter set and that cu_qp_delta_abs can be present in transform unit syntax. cu_qp_delta_enabled_flag equal to 0 specifies that the diff_cu_qp_delta_depth syntax element is not present in the picture parameter set and that cu_qp_delta_abs is not present in transform unit syntax."

comment:3 Changed 9 years ago by bbross

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

Will be fixed in JCTVC-K0030_v3 (to be released)

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)
  • Woo-Jin Han(Subscriber)