| Opened 14 years ago Closed 14 years ago #174 closed defect (fixed)MVP index bits not accounted for when comparing candidates in xCheckBestMVP
 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)Change History (5)Changed 14 years ago by nsprljancomment:1 Changed 14 years ago by fbossen
 
Fixed in r1009 comment:2 Changed 14 years ago by davidf
 
Updating component after adding WD (Text) tickets comment:3 Changed 14 years ago by davidf
 comment:4 Changed 14 years ago by davidf
 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 
 | ||||||||||||||||

Patched based on origin/tags/HM-3.1