[x265] [PATCH] param: Disable opt-qp-pps and opt-ref-list-length-pps by default

[Pradeep Ramachandran]

On Thu, Nov 16, 2017 at 10:56 AM, Pradeep Ramachandran <
pradeep at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Pradeep Ramachandran <pradeep at multicorewareinc.com>
> # Date 1510808150 -19800
> #      Thu Nov 16 10:25:50 2017 +0530
> # Node ID 8b12db6c5f6bbab87887822ae9fce524e18a25c1
> # Parent  dd9772385d152528201d335efbc6f75fdc43b08c
> param: Disable opt-qp-pps and opt-ref-list-length-pps by default
>
> The bits saved by these options is very miniscule, and not observable in
> most
> tests focusing on low bitrates, and low resolutions; this was the goal of
> these
> two options. Also, since these options change the PPS, the muxer has to
> ensure
> that the headers are sent every frame which seems to trouble some muxers
> and
> users.
>

Pushed to default branch


>
> diff -r dd9772385d15 -r 8b12db6c5f6b doc/reST/cli.rst
> --- a/doc/reST/cli.rst  Tue Nov 14 11:00:09 2017 +0530
> +++ b/doc/reST/cli.rst  Thu Nov 16 10:25:50 2017 +0530
> @@ -2159,12 +2159,12 @@
>  .. option:: --opt-qp-pps, --no-opt-qp-pps
>
>         Optimize QP in PPS (instead of default value of 26) based on the
> QP values
> -       observed in last GOP. Default enabled.
> +       observed in last GOP. Default disabled.
>
>  .. option:: --opt-ref-list-length-pps, --no-opt-ref-list-length-pps
>
>         Optimize L0 and L1 ref list length in PPS (instead of default
> value of 0)
> -       based on the lengths observed in the last GOP. Default enabled.
> +       based on the lengths observed in the last GOP. Default disabled.
>
>  .. option:: --multi-pass-opt-rps, --no-multi-pass-opt-rps
>
> diff -r dd9772385d15 -r 8b12db6c5f6b source/common/param.cpp
> --- a/source/common/param.cpp   Tue Nov 14 11:00:09 2017 +0530
> +++ b/source/common/param.cpp   Thu Nov 16 10:25:50 2017 +0530
> @@ -271,8 +271,8 @@
>
>      param->bEmitVUITimingInfo   = 1;
>      param->bEmitVUIHRDInfo      = 1;
> -    param->bOptQpPPS            = 1;
> -    param->bOptRefListLengthPPS = 1;
> +    param->bOptQpPPS            = 0;
> +    param->bOptRefListLengthPPS = 0;
>      param->bOptCUDeltaQP        = 0;
>      param->bAQMotion = 0;
>      param->bHDROpt = 0;
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20171116/44ce00b6/attachment.html>