It just occurred to me that if you hardcoded this to 6, you would be
penalising for ctu 32, 16.

On Mon, Mar 28, 2016 at 6:56 PM, Divya Manivannan <
[email protected]> wrote:

> # HG changeset patch
> # User Divya Manivannan <[email protected]>
> # 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.
>          else
>              rowCount = X265_MIN(m_refLagRows, m_numRows - 1);
>          if (row == rowCount)
> _______________________________________________
> x265-devel mailing list
> [email protected]
> https://mailman.videolan.org/listinfo/x265-devel
>



-- 
Deepthi Nandakumar
Engineering Manager, x265
Multicoreware, Inc
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to