Opened 4 years ago
Closed 4 years ago
#1461 closed defect (fixed)
Enabling chroma dQP does not exercise the feature as expected
Reported by: | delagrangep | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | VTM-12.1 |
Component: | VTM | Version: | VTM-11.2 |
Keywords: | Cc: | ksuehring, XiangLi, fbossen, jvet@… |
Description
When enabled, the chroma dQP index is exercised in EncCu::xCompressCU, by scanning all possible index values, depending on CU position. The local offset is stored in m_cuChromaQpOffsetIdxPlus1.
The value is set only when possible, i.e. when the CU is the first of a chroma QG. However just before checking that condition, m_cuChromaQpOffsetIdxPlus1 is always reset to zero. The consequence is a spurious reset of m_cuChromaQpOffsetIdxPlus1 for any QG that contains several CUs, defeating the purpose of the feature.
The solution is to move the m_cuChromaQpOffsetIdxPlus1=0 statement to an else branch of above-mentioned condition.
Change history (1)
comment:1 Changed 4 years ago by fbossen
- Milestone set to VTM-12.1
- Resolution set to fixed
- Status changed from new to closed
https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/-/merge_requests/2018