Opened 10 years ago

Closed 6 years ago

#398 closed enhancement (fixed)

Text / HM mismatch for derivation of TMVP in AMVP of HM-6.0rc1

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

Description

As described in section 8.5.2.1.5, the 2nd step of the ordered steps of deriving the motion vector predictor mvpLX:

If both availableFlagLXA and availableFlagLXB are equal to 1 and mvLXA is not equal to mvLXB, availableFlagLXCol is set equal to 0, otherwise, the derivation process for temporal luma motion vector prediction in subclause 5 is invoked with luma location ( xP, yP ).

It seems that we should append a checking before deriving the TMVP candidate in the function "fillMvpCand" of the file "TComDataCU.cpp" of HM6.0-rc1 as follows:

if (pInfo->iN < AMVP_MAX_NUM_CANDS)
{
if ( getSlice()->getPPS()->getEnableTMVPFlag() )
{
........
}
}

In this way, we can eliminate the unnecessary burden of deriving TMVP.

After deriving the TMVP candidate, we can delete the following statement, because it is impossible to satisfy the judgment.
if (pInfo->iN > AMVP_MAX_NUM_CANDS)

{

pInfo->iN = AMVP_MAX_NUM_CANDS;

}

Change History (3)

comment:1 Changed 10 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 10 years ago by ksuehring

  • Version changed from HM-6.0rc1 to HM-6.0

comment:3 Changed 6 years ago by karlsharman

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

Addressed in r4597.

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)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)
  • NaZhang(Reporter)