#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@… |
Description
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 4 years ago by bbross
comment:2 Changed 4 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 4 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.
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.