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. > 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 > [email protected] > https://mailman.videolan.org/listinfo/x265-devel >
_______________________________________________ x265-devel mailing list [email protected] https://mailman.videolan.org/listinfo/x265-devel
