Opened 11 years ago

Closed 11 years ago

#28 closed defect (fixed)

encoder bug in motion search for ILR (non-CTC)

Reported by: bugdayci Owned by: Vadim
Priority: minor Milestone: SHM-6.1
Component: SHM software Version: SHM-6.0
Keywords: Cc: Vadim, jct-vc@…

Description

In a 4 layered scenario where layer 1 depends on layer 0, layer 2 is independent and layer 3 depends on layer 2 so that a decoder can skip layers 0 and 1 and start decoding from layer 3, encoder does not work properly. When I checked the code, I realized that layer 3 does not refer to layer 2 in mode decision due to a bug in the predInterSearchILRUni function. The bugfix may be to change the line number 6013 in TEncSearch.cpp in SHM-6.0 as follows:

if( pcCU->getSlice()->getRefPic(eRefPicList, refIdx)->isILR(pcCU->getLayerId())) && pcCU->getSlice()->getRefPic(eRefPicList, refIdx)->getLayerId() == refLayerId )

I removed the last part where the reference layer Id is compared to refLayerId variable which is obtained from pcSlice->getInterLayerPredLayerIdc(refLayer). (reference layer Id is 2 but the InterLayerPredLayerIdc is 0)

I attached a configuration file where you can duplicate the problem and two encoder log files for SHM-6.0 and bugfix.

Attachments (3)

encoder_randomaccess_main.cfg (17.4 KB) - added by bugdayci 11 years ago.
enc_with_bug.log (6.2 KB) - added by bugdayci 11 years ago.
enc_with_bugfix.log (6.2 KB) - added by bugdayci 11 years ago.

Download all attachments as: .zip

Change history (5)

comment:1 Changed 11 years ago by DefaultCC Plugin

  • Cc Vadim jct-vc@… added

Changed 11 years ago by bugdayci

Changed 11 years ago by bugdayci

Changed 11 years ago by bugdayci

comment:2 Changed 11 years ago by Vadim

  • Milestone changed from SHM-6.0 to SHM-6.1
  • Resolution set to fixed
  • Status changed from new to closed

fixed with rev 768, 769

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

  • Done Bugdayci(Reporter)
  • Gerhard Tech(Always)
  • jct-vc@…(Subscriber)
  • Karsten Suehring(Always)
  • Vadim Seregin(Owner, Subscriber, Participant)