<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Nov 16, 2017 at 10:56 AM, Pradeep Ramachandran <span dir="ltr"><<a href="mailto:pradeep@multicorewareinc.com" target="_blank">pradeep@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 Pradeep Ramachandran <<a href="mailto:pradeep@multicorewareinc.com">pradeep@multicorewareinc.com</a>><br>
# Date 1510808150 -19800<br>
#      Thu Nov 16 10:25:50 2017 +0530<br>
# Node ID 8b12db6c5f6bbab87887822ae9fce5<wbr>24e18a25c1<br>
# Parent  dd9772385d152528201d335efbc6f7<wbr>5fdc43b08c<br>
param: Disable opt-qp-pps and opt-ref-list-length-pps by default<br>
<br>
The bits saved by these options is very miniscule, and not observable in most<br>
tests focusing on low bitrates, and low resolutions; this was the goal of these<br>
two options. Also, since these options change the PPS, the muxer has to ensure<br>
that the headers are sent every frame which seems to trouble some muxers and<br>
users.<br></blockquote><div><br></div><div>Pushed to default branch</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
diff -r dd9772385d15 -r 8b12db6c5f6b doc/reST/cli.rst<br>
--- a/doc/reST/cli.rst  Tue Nov 14 11:00:09 2017 +0530<br>
+++ b/doc/reST/cli.rst  Thu Nov 16 10:25:50 2017 +0530<br>
@@ -2159,12 +2159,12 @@<br>
 .. option:: --opt-qp-pps, --no-opt-qp-pps<br>
<br>
        Optimize QP in PPS (instead of default value of 26) based on the QP values<br>
-       observed in last GOP. Default enabled.<br>
+       observed in last GOP. Default disabled.<br>
<br>
 .. option:: --opt-ref-list-length-pps, --no-opt-ref-list-length-pps<br>
<br>
        Optimize L0 and L1 ref list length in PPS (instead of default value of 0)<br>
-       based on the lengths observed in the last GOP. Default enabled.<br>
+       based on the lengths observed in the last GOP. Default disabled.<br>
<br>
 .. option:: --multi-pass-opt-rps, --no-multi-pass-opt-rps<br>
<br>
diff -r dd9772385d15 -r 8b12db6c5f6b source/common/param.cpp<br>
--- a/source/common/param.cpp   Tue Nov 14 11:00:09 2017 +0530<br>
+++ b/source/common/param.cpp   Thu Nov 16 10:25:50 2017 +0530<br>
@@ -271,8 +271,8 @@<br>
<br>
     param->bEmitVUITimingInfo   = 1;<br>
     param->bEmitVUIHRDInfo      = 1;<br>
-    param->bOptQpPPS            = 1;<br>
-    param->bOptRefListLengthPPS = 1;<br>
+    param->bOptQpPPS            = 0;<br>
+    param->bOptRefListLengthPPS = 0;<br>
     param->bOptCUDeltaQP        = 0;<br>
     param->bAQMotion = 0;<br>
     param->bHDROpt = 0;<br>
</blockquote></div><br></div></div>