[x265] [PATCH] rc: Calculate the variance data for weighted prediction if aq-mode is disable

[Deepthi Nandakumar]

What about the variance value returned by acEnergyCu?


On Mon, Dec 16, 2013 at 10:43 PM, Deepthi Nandakumar <
deepthi at multicorewareinc.com> wrote:

> Ughh - there's something wrong with my hg import. All clear now.
>
>
> On Mon, Dec 16, 2013 at 10:34 PM, Deepthi Nandakumar <
> deepthi at multicorewareinc.com> wrote:
>
>> Am I missing something here? You have added code in the else part - if(AQ
>> is disabled), we are already calculating variance for weightP.
>>
>>
>> On Mon, Dec 16, 2013 at 5:43 PM, Gopu Govindaswamy <
>> gopu at multicorewareinc.com> wrote:
>>
>>> # HG changeset patch
>>> # User Gopu Govindaswamy <gopu at multicorewareinc.com>
>>> # Date 1387195927 -19800
>>> # Node ID bdc7afa5ba5afee7113ac0d427fbfc660359716d
>>> # Parent  3dae450a06a4b390ce6fd947d5095b739b01c6d8
>>> rc: Calculate the variance data for weighted prediction if aq-mode is
>>> disable
>>>
>>> diff -r 3dae450a06a4 -r bdc7afa5ba5a source/encoder/ratecontrol.cpp
>>> --- a/source/encoder/ratecontrol.cpp    Mon Dec 16 13:57:58 2013 +0530
>>> +++ b/source/encoder/ratecontrol.cpp    Mon Dec 16 17:42:07 2013 +0530
>>> @@ -116,6 +116,15 @@
>>>              for (int cuxy = 0; cuxy < cuCount; cuxy++ )
>>>                  pic->m_lowres.invQscaleFactor[cuxy] = 256;
>>>          }
>>> +
>>> +         /* Need variance data for weighted prediction */
>>> +        if (cfg->param.bEnableWeightedPred)
>>> +        {
>>> +            for (int cuy = 0; cuy < maxRow; cuy++ )
>>> +                for (int cux = 0; cux < maxCol; cux++ )
>>> +                    acEnergyCu(pic, cux, cuy);
>>> +        }
>>> +
>>>      }
>>>      else
>>>      {
>>> _______________________________________________
>>> x265-devel mailing list
>>> x265-devel at videolan.org
>>> https://mailman.videolan.org/listinfo/x265-devel
>>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20131216/b134e012/attachment.html>