<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 3, 2013 at 12:58 AM, Deepthi Nandakumar <span dir="ltr"><<a href="mailto:deepthi@multicorewareinc.com" target="_blank">deepthi@multicorewareinc.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>This patch essentially reverts a merge optimization that was pushed a few weeks ago. <br>
<br></div>The merge optimization included removing a cost estimate (merge mode with residual) from consideration. This caused a bitrate/quality drop in rd 0/1 - this penalty needs to be investigated more before it becomes either a user option or part of rd 0/1. </div>
</blockquote><div><br></div><div>I did a quick test with "x265 \mcw\city_4cif.y4m out.hevc -F2 --rd=0" on my slow laptop:</div><div><br></div><div>current tip:</div><div> encoded 600 frames in 177.32s (3.38 fps), 515.63 kb/s, Global PSNR: 34.654</div>
<div><br></div><div>this patch applied:</div><div><div> encoded 600 frames in 244.51s (2.45 fps), 506.49 kb/s, Global PSNR: 34.611</div></div><div><br></div><div>Not a very compelling change at first glance; I'm inclined to leave the current behavior until there is an unambiguous improvement or if you have better circumstantial evidence than my quick tests.</div>
<div><br></div></div>-- <br>Steve Borho
</div></div>