[x265] [PATCH] presets update

[Pradeep Ramachandran]

On Tue, Jun 21, 2016 at 11:07 AM, <mahesh at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Mahesh Pittala <mahesh at multicorewareinc.com>
> # Date 1464261309 -19800
> #      Thu May 26 16:45:09 2016 +0530
> # Node ID eb8d8a1b421c1ee9e442eb7d69fde8365f0766c8
> # Parent  78ffb67a844e3e76facf18c52790f1bd544754d6
> presets update
>
> during NAB we tuned faster presets for better speed and quality, observed
> quality
> (SSIM) metrics are improved at same speed with these setings after applying
> "rd determine CU complexity to skip analysis of higher depths" patch.
> visual quality also improved a bit.
>
> diff -r 78ffb67a844e -r eb8d8a1b421c doc/reST/presets.rst
> --- a/doc/reST/presets.rst      Fri Jun 10 15:53:28 2016 +0530
> +++ b/doc/reST/presets.rst      Thu May 26 16:45:09 2016 +0530
> @@ -49,11 +49,11 @@
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
>  | scenecut        |  0  | 40  | 40  |  40 |  40 |  40 |  40  |  40  |
> 40  | 40   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
> -| ref             |  1  |  1  |  2  |   2 |   3 |   3 |   4  |   4  |
>  5  |  5   |
> +| ref             |  2  |  2  |  2  |   2 |   3 |   3 |   4  |   4  |
>  5  |  5   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
> -| limit-refs      |  0  |  0  |  3  |   3 |   3 |   3 |   3  |   2  |
>  1  |  0   |
> +| limit-refs      |  3  |  3  |  3  |   3 |   3 |   3 |   3  |   2  |
>  1  |  0   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
> -| me              | dia | hex | hex | hex | hex | hex | star | star |
> star | star |
> +| me              | hex | hex | hex | hex | hex | hex | star | star |
> star | star |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
>  | merange         | 57  | 57  | 57  |  57 |  57 |  57 |  57  |  57  |
> 57  | 92   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
> @@ -73,11 +73,11 @@
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
>  | fast-intra      |  1  |  1  |  1  |   1 |   1 |   0 |   0  |   0  |
>  0  |  0   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
> -| b-intra         |  0  |  0  |  0  |   0 |   0 |   0 |   0  |   1  |
>  1  |  1   |
> +| b-intra         |  1  |  1  |  1  |   0 |   0 |   0 |   0  |   1  |
>  1  |  1   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
>  | sao             |  0  |  0  |  1  |   1 |   1 |   1 |   1  |   1  |
>  1  |  1   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
> -| signhide        |  0  |  1  |  1  |   1 |   1 |   1 |   1  |   1  |
>  1  |  1   |
> +| signhide        |  1  |  1  |  1  |   1 |   1 |   1 |   1  |   1  |
>  1  |  1   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
>

This line can now be removed from the presets table as it is the same
across all presets.


>  | weightp         |  0  |  0  |  1  |   1 |   1 |   1 |   1  |   1  |
>  1  |  1   |
>
>  +-----------------+-----+-----+-----+-----+-----+-----+------+------+------+------+
> diff -r 78ffb67a844e -r eb8d8a1b421c source/common/param.cpp
> --- a/source/common/param.cpp   Fri Jun 10 15:53:28 2016 +0530
> +++ b/source/common/param.cpp   Thu May 26 16:45:09 2016 +0530
> @@ -275,14 +275,15 @@
>              param->bframes = 3;
>              param->bFrameAdaptive = 0;
>              param->subpelRefine = 0;
> -            param->searchMethod = X265_DIA_SEARCH;
> +            param->searchMethod = X265_HEX_SEARCH;
>              param->bEnableEarlySkip = 1;
>              param->bEnableSAO = 0;
> -            param->bEnableSignHiding = 0;
> +            param->bEnableSignHiding = 1;
>

This is the same as this is the same as the default value.

             param->bEnableWeightedPred = 0;
>              param->rdLevel = 2;
> -            param->maxNumReferences = 1;
> -            param->limitReferences = 0;
> +            param->maxNumReferences = 2;
> +            param->limitReferences = 3;
>

Same as default value - can be removed


> +            param->bIntraInBFrames = 1;
>              param->rc.aqStrength = 0.0;
>              param->rc.aqMode = X265_AQ_NONE;
>              param->rc.qgSize = 32;
> @@ -298,8 +299,9 @@
>              param->bEnableEarlySkip = 1;
>              param->bEnableWeightedPred = 0;
>              param->rdLevel = 2;
> -            param->maxNumReferences = 1;
> -            param->limitReferences = 0;
> +            param->maxNumReferences = 2;
> +            param->limitReferences = 3;
>

Same as default value - can be removed


> +            param->bIntraInBFrames = 1;
>              param->rc.aqStrength = 0.0;
>              param->rc.aqMode = X265_AQ_NONE;
>              param->rc.qgSize = 32;
> @@ -314,6 +316,7 @@
>              param->bEnableEarlySkip = 1;
>              param->rdLevel = 2;
>              param->maxNumReferences = 2;
> +            param->bIntraInBFrames = 1;
>              param->rc.qgSize = 32;
>              param->bEnableFastIntra = 1;
>          }
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20160621/1f5901db/attachment-0001.html>