Opened 10 years ago

Closed 10 years ago

#174 closed defect (fixed)

MVP index bits not accounted for when comparing candidates in xCheckBestMVP

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

Description

In function TEncSearch::xCheckBestMVP, when checking the cost of each available MV predictor, the cost of signalling the index (specified in the array m_auiMVPIdxCost) is not taken into account.

The results with the fix applied, based on 24 pictures and obtained with HM-3.0-dev are as follows:

         Random access              Random access LoCo
         Y        U        V        Y        U        V
Class A  0.0      -0.2     -0.2     0.0      -0.1     -0.1
Class B  0.0      0.0      -0.2     -0.1     -0.1     -0.1
Class C  0.0      -0.1     0.2      0.0      0.0      -0.1
Class D  -0.1     -0.2     -0.2     -0.1     0.1      -0.2
Class E
All      0.0      -0.1     -0.1     0.0      0.0      -0.1

         Low delay                  Low delay LoCo
         Y        U        V        Y        U        V
Class A
Class B  -0.1     -0.2     0.0      0.0      0.1      0.1
Class C  0.0      0.3      0.2      0.0      0.2      0.2
Class D  0.0      -0.1     0.4      -0.1     -0.3     -0.1
Class E  -0.1     -0.2     -0.3     0.0      0.2      0.1
All      0.0      0.0      0.1      0.0      0.0      0.0

         Low delay (P)              Low delay LC (P)
         Y        U        V        Y        U        V
Class A
Class B  0.0      0.3      0.3      0.0      -0.1     -0.1
Class C  0.0      -0.1     -0.1     0.0      -0.1     -0.1
Class D  -0.2     0.1      0.1      0.0      0.1      0.1
Class E  -0.1     -0.4     -0.4     0.0      0.4      0.4
All      -0.1     -0.1     -0.1     0.0      0.0      0.0 

Attachments (1)

bugfix174.patch (1.3 KB) - added by nsprljan 10 years ago.
Patched based on origin/tags/HM-3.1

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by nsprljan

Patched based on origin/tags/HM-3.1

comment:1 Changed 10 years ago by fbossen

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

Fixed in r1009

comment:2 Changed 10 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:3 Changed 9 years ago by davidf

  • Cc fbossen ksuehring davidf added

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