<div dir="ltr">It just occurred to me that if you hardcoded this to 6, you would be penalising for ctu 32, 16. </div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 28, 2016 at 6:56 PM, Divya Manivannan <span dir="ltr"><<a href="mailto:divya@multicorewareinc.com" target="_blank">divya@multicorewareinc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Divya Manivannan <<a href="mailto:divya@multicorewareinc.com">divya@multicorewareinc.com</a>><br>
# Date 1459171242 -19800<br>
#      Mon Mar 28 18:50:42 2016 +0530<br>
# Node ID 2f03f22faf370fe5f70c16a2f1d021613ac84851<br>
# Parent  5dbd6a0c8e17481a0c4d31243ebc8b46ad59e15d<br>
rc: change the number of rows for updating rc stats<br>
<br>
diff -r 5dbd6a0c8e17 -r 2f03f22faf37 source/encoder/frameencoder.cpp<br>
--- a/source/encoder/frameencoder.cpp   Mon Mar 28 12:53:40 2016 +0530<br>
+++ b/source/encoder/frameencoder.cpp   Mon Mar 28 18:50:42 2016 +0530<br>
@@ -1214,7 +1214,7 @@<br>
     if (m_param->rc.rateControlMode == X265_RC_ABR || bIsVbv)<br>
     {<br>
         if ((uint32_t)m_rce.encodeOrder <= 2 * (m_param->fpsNum / m_param->fpsDenom))<br>
-            rowCount = X265_MIN((m_numRows + 1) / 2, m_numRows - 1);<br>
+            rowCount = X265_MIN((m_numRows + 1) / 2, 6); // 6 is used to avoid updating stats after half the frame in 1080p, 4k videos.<br>
         else<br>
             rowCount = X265_MIN(m_refLagRows, m_numRows - 1);<br>
         if (row == rowCount)<br>
_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div>Deepthi Nandakumar<br></div>Engineering Manager, x265<br></div>Multicoreware, Inc<br></div></div>
</div>