<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 14, 2016 at 7:20 PM,  <span dir="ltr"><<a href="mailto:bhavna@multicorewareinc.com" target="_blank">bhavna@multicorewareinc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"># HG changeset patch<br>
# User Bhavna Hariharan <<a href="mailto:bhavna@multicorewareinc.com">bhavna@multicorewareinc.com</a>><br>
# Date 1476275329 -19800<br>
#      Wed Oct 12 17:58:49 2016 +0530<br>
# Node ID 854149baceefa075c3b1af12433680<wbr>ffda2e3b64<br>
# Parent  c97805dad9148ad3cddba10a67ed55<wbr>96508e8f86<br>
limitTU: fix energy calculation used in limiting TU recursion<br>
<br>
This commit changes the output of limit TU<br>
<br>
diff -r c97805dad914 -r 854149baceef source/encoder/search.cpp<br>
--- a/source/encoder/search.cpp Thu Oct 13 17:53:48 2016 +0800<br>
+++ b/source/encoder/search.cpp Wed Oct 12 17:58:49 2016 +0530<br>
@@ -3420,14 +3420,15 @@<br>
         if (m_param->limitTU && bCheckSplit)<br>
         {<br>
             // Stop recursion if the TU's energy level is minimal<br>
+            uint32_t numCoeff = trSize * trSize;<br>
             if (cbfFlag[TEXT_LUMA][0] == 0)<br>
                 bCheckSplit = false;<br>
-            else if (numSig[TEXT_LUMA][0] < (cuGeom.numPartitions / 16))<br>
+            else if (numSig[TEXT_LUMA][0] < (numCoeff / 64))<br>
             {<br>
                 uint32_t energy = 0;<br>
-                for (uint32_t i = 0; i < cuGeom.numPartitions; i++)<br>
+                for (uint32_t i = 0; i < numCoeff; i++)<br>
                     energy += abs(coeffCurY[i]);<br>
-                if (energy < numSig[TEXT_LUMA][0])<br>
+                if (energy == numSig[TEXT_LUMA][0])<br>
                     bCheckSplit = false;<br></div></div></blockquote><div><br></div><div>Can you give an example where CheckSplit is disabled here? I am finding it hard to reason conditions under which this condition is satisfied.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
             }<br>
         }<br>
</div></div>______________________________<wbr>_________________<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/<wbr>listinfo/x265-devel</a><br>
</blockquote></div><br></div></div>