<div dir="ltr">Hmm, this might reveal a bug. addSplitCost is called for splitPred after the 4 sub-blocks have been evaluated. Why are we adding this cost again, especially for md.bestMode (which could be a non-split mode)?</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 22, 2016 at 5:10 AM,  <span dir="ltr"><<a href="mailto:sagar@multicorewareinc.com" target="_blank">sagar@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 Sagar Kotecha<<a href="mailto:sagar@multicorewareinc.com">sagar@multicorewareinc.com</a>><br>
# Date 1461327009 -19800<br>
#      Fri Apr 22 17:40:09 2016 +0530<br>
# Node ID 3f2dfc9dad6e8756c4a9b5afd5b5f38516e04570<br>
# Parent  02d79be487d7f825c961d15535a8681a201da3b1<br>
analysis: do not add split cost if skip is chosen<br>
<br>
diff -r 02d79be487d7 -r 3f2dfc9dad6e source/encoder/analysis.cpp<br>
--- a/source/encoder/analysis.cpp       Sun Apr 17 21:07:28 2016 +0000<br>
+++ b/source/encoder/analysis.cpp       Fri Apr 22 17:40:09 2016 +0530<br>
@@ -1329,7 +1329,7 @@<br>
         if (m_bTryLossless)<br>
             tryLossless(cuGeom);<br>
<br>
-        if (mightSplit)<br>
+        if (mightSplit && !foundSkip)<br>
             addSplitFlagCost(*md.bestMode, cuGeom.depth);<br>
     }<br>
<br>
@@ -1707,7 +1707,7 @@<br>
         if (m_bTryLossless)<br>
             tryLossless(cuGeom);<br>
<br>
-        if (mightSplit)<br>
+        if (mightSplit && !foundSkip)<br>
             addSplitFlagCost(*md.bestMode, cuGeom.depth);<br>
     }<br>
<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>