Opened 5 years ago
Closed 5 years ago
#769 closed enhancement (fixed)
Not matched names between pps_joint_cbcr_qp_offset_value in syntax table/semantics and pps_joint_cbcr_qp_offset in eq.(1124)
Reported by: | tsukuba.takeshi | Owned by: | |
---|---|---|---|
Priority: | trivial | Milestone: | |
Component: | spec | Version: | |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
In WD (JVET-P0041), pps_joint_cbcr_qp_offset_value is defined in PPS as below:
pps_joint_cbcr_qp_offset_value specifies the offset to the luma quantization parameter Qp′Y used for deriving Qp′CbCr. The value of pps_joint_cbcr_qp_offset_value shall be in the range of −12 to +12, inclusive. When ChromaArrayType is equal to 0 or sps_joint_cbcr_enabled_flag is equal to 0, pps_joint_cbcr_qp_offset_value is not used in the decoding process and decoders shall ignore its value. When pps_joint_cbcr_qp_offset_present_flag is equal to 0, pps_joint_cbcr_qp_offset_value is not present and is inferred to be equal to 0.
On the other hand, pps_joint_cbcr_qp_offset is used in eq.(1124) in sec.8.7.1 as below:
Qp′CbCr = Clip3( −QpBdOffset, 63, qPCbCr + pps_joint_cbcr_qp_offset +
slice_joint_cbcr_qp_offset +CuQpOffsetCbCr ) + QpBdOffset (1124)
For minimum change, pps_joint_cbcr_qp_offset should be replaced with pps_joint_cbcr_qp_offset_value.
Or,
pps_joint_cbcr_qp_offset_value in syntax table and semantics should be replced with pps_joint_cbcr_qp_offset.
Change history (1)
comment:1 Changed 5 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Thanks for reporting!
This has been fixed in JVET-Q2001-v5.