[x265] [PATCH] rc: change the number of rows for updating rc stats

[Pradeep Ramachandran]

On Mon, Mar 28, 2016 at 6:56 PM, Divya Manivannan <
divya at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Divya Manivannan <divya at multicorewareinc.com>
> # Date 1459171242 -19800
> #      Mon Mar 28 18:50:42 2016 +0530
> # Node ID 2f03f22faf370fe5f70c16a2f1d021613ac84851
> # Parent  5dbd6a0c8e17481a0c4d31243ebc8b46ad59e15d
> rc: change the number of rows for updating rc stats
>
> diff -r 5dbd6a0c8e17 -r 2f03f22faf37 source/encoder/frameencoder.cpp
> --- a/source/encoder/frameencoder.cpp   Mon Mar 28 12:53:40 2016 +0530
> +++ b/source/encoder/frameencoder.cpp   Mon Mar 28 18:50:42 2016 +0530
> @@ -1214,7 +1214,7 @@
>      if (m_param->rc.rateControlMode == X265_RC_ABR || bIsVbv)
>      {
>          if ((uint32_t)m_rce.encodeOrder <= 2 * (m_param->fpsNum /
> m_param->fpsDenom))
> -            rowCount = X265_MIN((m_numRows + 1) / 2, m_numRows - 1);
> +            rowCount = X265_MIN((m_numRows + 1) / 2, 6); // 6 is used to
> avoid updating stats after half the frame in 1080p, 4k videos.
>

Would it then be a better idea to check for resolution and set the # rows
to 6 only for 4K and 1080p videos, and let it to be m_numRows - 1 for the
other resolutions?


>          else
>              rowCount = X265_MIN(m_refLagRows, m_numRows - 1);
>          if (row == rowCount)
> _______________________________________________
> 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/20160328/5da873c7/attachment.html>