Opened 5 years ago

Last modified 4 years ago

#572 new defect

Incorrect use of chroma distortion weights if Cb and Cr QP offsets differ (affects perceptual QPA, may affect SBT in HDR-PQ CTCs)

Reported by: crhelmrich Owned by:
Priority: major Milestone:
Component: VTM Version: VTM-6.1
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

VTM 6.x, in initEncSlice(), calls setUpLambda() to initialize the Cb and Cr distortion weights. These weights are computed, and multiplied onto the respective component distortions in RdCost::getDistPart(), independently for Cb and Cr. However, there are some places where an average chroma weight is calculated via RdCost::getChromaWeight() and used for both Cb and Cr. This does not seem to be an issue with the SDR or HDR-HLG CTCs, but if I understand correctly, different Cb and Cr QP offsets (and, thus, different Cb and Cr lambdas) may occur in the HDR-PQ CTCs.

Therefore, in the case of unequal Cb and Cr QP offsets, the following encoder functions may, according to my understanding, work suboptimally:

  • rate control (if pCfg->getUseRateCtrl() in encodeCtus(), not sure if this is used)
  • perceptual QPA with chroma QP adaptation (e.g., if pCfg->getUsePerceptQPA() && pcSlice->getPPS()->getUseDQP() in encodeCtus())
  • SBT fast algorithm (InterSearch::calcMinDistSbt() called in xEncodeInterResidual()).

Before I submit a merge request with a proposed fix (used of a RdCost::getDistortionWeight(compID) instead of the RdCost::getChromaWeight() at the respective places), it would be great if someone could confirm that my understanding is correct.

Thanks and best,

Christian

Attachments (1)

hdr__vtm6_vs_vtm6withFix572.xlsm (103.9 KB) - added by crhelmrich 5 years ago.
check of the effect of this fix on HDR CTCs

Download all attachments as: .zip

Change history (3)

Changed 5 years ago by crhelmrich

check of the effect of this fix on HDR CTCs

comment:1 Changed 5 years ago by crhelmrich

Since I did not receive any comments on this, I went ahead and created merge request
https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/1058
to fix the issue for the unequal Cb/Cr QP offsets and a related combination-with-JCCR case.

Note that I did NOT fix the SBT case since the fix would affect the HDR class 1 CTCs. See the attached Excel file for details, the results don't seem to show a benefit by fixing the InterSearch::calcMinDistSbt() function used by SBT.

Best,

Christian

comment:2 Changed 4 years ago by ksuehring

  • Milestone VTM-6.2 deleted
Note: See TracTickets for help on using tickets.