Opened 10 years ago

Closed 10 years ago

#146 closed defect (fixed)

Bug in HM3.0 in the implementation of spatial mvp scaling.

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

Description

Bug in HM3.0 in the implementation of spatial mvp scaling.

In TComDataCU::fillMvpCand() around line #2893:


if(!bAdded)
{

bAdded = xAddMVPCand( pInfo, eRefPicList, iRefIdx, uiPartIdxLT, MD_ABOVE_LEFT);
if (bAdded && pInfo->iN==2 && pInfo->m_acMvCand[0] == pInfo->m_acMvCand[1])
{

pInfo->iN--; remove duplicate entries

bAdded = false; <--- This line should be added

}

}

Without this statement, bAdded remains true although mv is not added to the list. Therefore,

MTK_AMVP_SMVP_DERIVATION code is not performed in this case.

Change History (5)

comment:1 Changed 10 years ago by madhukar

  • Version set to HM-3.0

comment:2 Changed 10 years ago by davidf

  • Milestone set to HM-3.1
  • Resolution set to fixed
  • Status changed from new to closed

This was fixed in r828

comment:3 Changed 9 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:4 Changed 9 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:5 Changed 9 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

  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)
  • Madhukar Budagavi(Reporter, Participant)