Opened 13 years ago

Closed 13 years ago

#207 closed defect (fixed)

HM4.0 encoder issue on chroma ALF for multiple slice coding

Reported by: yamakage Owned by:
Priority: minor Milestone:
Component: HM Version: HM-4.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

ALF coefficient design for Cr uses Cb information when coding multiple slices per picture. This does not affect coding results for the common test conditions (i.e., single slice per picture), but affects when multiple slice coding is enabled such as CE8.c.5 and CE8.c.6.

Suggested change is as follows:

In TEncAdaptiveLoopFilter.cpp, line 6130,

xCalcCorrelationFuncforChromaSlices(ALF_Cb, pOrg, pCmp, iShape, pcPicOrg->getCStride(), pcPicDec->getCStride());

should be modified to the following:

xCalcCorrelationFuncforChromaSlices(ALF_Cr, pOrg, pCmp, iShape, pcPicOrg->getCStride(), pcPicDec->getCStride());

(the first argument should be modified from ALF_Cb to ALF_Cr.)

Change History (4)

comment:1 Changed 13 years ago by chiayang_tsai

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

Fixed in r1395

comment:2 Changed 13 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:3 Changed 13 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:4 Changed 13 years ago by davidf

  • Cc jct-vc@… added
Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Chia-Yang Tsai(Participant)
  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)
  • Tomoo Yamakage(Reporter)