Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#654 closed defect (fixed)

Valid range for LADF QP Offsets

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D7 vB
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…


According to the semantics section, the sps_ladf_lowest_interval_qp_offset
and sps_ladf_qp_offset shall be in the range of 0 to 63, inclusive.

Is this range correct? If so, why are they coded as signed se(v)?

The VTM config files would appear to allow the range from -63 to +63. Perhaps that was the intended range?

Change history (3)

comment:1 Changed 5 years ago by bbross

In the adopted text provided in v5 of JVET-L0414 (JVET-L0414_draft_text-r2.docx), the coding was changed from ue(v) to se(v) and the range was changed from 0 to max QP value to min QP value to max QP value.
This and the VTM implementation indicate that the range should be from -63 to +63 as suggested by Brian.
However, I would like to have the proponents of LADF confirm that.

comment:2 Changed 5 years ago by bbross

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

Proponent confirmed and this will be fixed JVET-P2001-vC.

comment:3 Changed 5 years ago by bbross

Same range adjustment should be done for sps_ladf_lowest_interval_qp_offset which is se(v) coded as well.

Thanks Rémy for pointing this out.

Note: See TracTickets for help on using tickets.