<div dir="ltr"><div><div><div>About the presets themselves,<br><br></div>1. Whats the tradeoff from increased lookaheadDepth? Should fast/medium really use 30/40?<br></div>2. Can we remove atleast one of the "slow" presets? slower?? <br>
</div>3. EnableTSkipFast=1 for all but placebo preset. could be disabled in all medium and slow presets?<br><div><br><div><br><div><div><div id="__tbSetup"></div></div></div></div></div></div><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Thu, Nov 7, 2013 at 3:37 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><div><div><div><div>Since the default preset is medium, shouldnt x265_param_default apply medium settings only? So that the if (!strcmp(preset, "medium")) block here is empty?? <br></div>In that case, the preset (if specified) will change only the ones which are different from medium? <br>

<br></div>Also, there are a few params here that<br></div>1. arent used and arent initialised properly (eg, constrainedIntra, weightedBipred)  lets remove them and add them back when the feature exists. <br></div>2. that are dependent on other param fields, like EnableRdo, EnabledRDOQ depends on RDLevel. This particular case is handled safely inside encoder::configure, but not sure about all such cases. <br>

<br></div>Just thought this is a good time for springcleaning. <br></div>
</blockquote></div><br></div>