Opened 5 years ago

Closed 4 years ago

#144 closed defect (wontfix)

Initialization of List 1 motion vector is missing in getInterMMVDMergeCandidates(..) function.

Reported by: ss00.jeong Owned by:
Priority: minor Milestone:
Component: VTM Version:
Keywords: MMVD Cc: ksuehring, XiangLi, fbossen, jvet@…

Description (last modified by ksuehring)

The following code

  if (currBaseNum < MMVD_BASE_MV_NUM)
  {
    for (k = currBaseNum; k < MMVD_BASE_MV_NUM; k++)
    {
      mrgCtx.mmvdBaseMv[k][0] = MvField(Mv(0, 0), 0);
      mrgCtx.mmvdBaseMv[k][0] = MvField(Mv(0, 0), 0);
    }
  }

should be replaced by

  if (currBaseNum < MMVD_BASE_MV_NUM)
  {
    for (k = currBaseNum; k < MMVD_BASE_MV_NUM; k++)
    {
      mrgCtx.mmvdBaseMv[k][0] = MvField(Mv(0, 0), 0);
      mrgCtx.mmvdBaseMv[k][0] = MvField(Mv(0, 0), 0);

      if (pu.cu->slice->getSliceType() == B_SLICE)
      {
        mrgCtx.mmvdBaseMv[k][1] = MvField(Mv(0, 0), 0);
        mrgCtx.mmvdBaseMv[k][1] = MvField(Mv(0, 0), 0);
      }

    }
  }

.

There is no performance change.

Change history (2)

comment:1 Changed 5 years ago by ksuehring

  • Description modified (diff)

comment:2 Changed 4 years ago by fbossen

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

This doesn't seem to apply anymore.

Note: See TracTickets for help on using tickets.