Opened 9 years ago

Closed 7 years ago

#399 closed enhancement (fixed)

AMVP spatial scaling candidate in HM-6.0rc1

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

Description

As we all know, to remove the dependency of deriving the spatial candidate of upper position on left position, the simplification 2 of JCTVC-G542 defines a new derivation way of the parameter isScaledFlagLX and be adopted in the 7th meeting.

However, in the function "fillMvpCand" of the file "TComDataCU.cpp" of HM6.0-rc1, before the scaling calculation for upper position, there are two statements:

bAdded = bAddedSmvp;
if (pInfo->iN==2) bAdded = true;

In my opinion, the 2nd statement can be deleted, as we only can derive the value of pInfo->iN after checking all the left positions. As a result, we did not really eliminate the dependencies between derivation of the spatial candidate of upper position and left position.

After we carry out this modification, the experiment results is not affected, as the 2nd statement has no substantive role, whereas we really eliminate the dependency.

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by ksuehring

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

comment:3 Changed 7 years ago by fbossen

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

Fixed in r3778 r3779

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