[x265] [PATCH] CLI: add new option cu-min(minimum CU size)

[Steve Borho]

On 02/18, santhoshini at multicorewareinc.com wrote:
> # HG changeset patch
> # User Santhoshini Sekar<santhoshini at multicorewareinc.com>
> # Date 1424075543 -19800
> #      Mon Feb 16 14:02:23 2015 +0530
> # Node ID d0e4f7ae7d85b1d733d718fa557ab62a9c99861f
> # Parent  21257215be0f5fe861f83a6f767a9741012cdf50
> CLI: add new option cu-min(minimum CU size)

queued, with some additional detail in reST about what the setting is
expected to do.

> diff -r 21257215be0f -r d0e4f7ae7d85 doc/reST/cli.rst
> --- a/doc/reST/cli.rst	Wed Feb 18 16:20:17 2015 +0530
> +++ b/doc/reST/cli.rst	Mon Feb 16 14:02:23 2015 +0530
> @@ -477,6 +477,10 @@
>  	and less frame parallelism as well. Because of this the faster
>  	presets use a CU size of 32. Default: 64
>  
> +.. option:: --min-cu-size, <64|32|16|8>
> +
> +	Minimum CU size (width and height). Default: 8
> +
>  .. option:: --rect, --no-rect
>  
>  	Enable analysis of rectangular motion partitions Nx2N and 2NxN
> diff -r 21257215be0f -r d0e4f7ae7d85 source/CMakeLists.txt
> --- a/source/CMakeLists.txt	Wed Feb 18 16:20:17 2015 +0530
> +++ b/source/CMakeLists.txt	Mon Feb 16 14:02:23 2015 +0530
> @@ -21,7 +21,7 @@
>  include(CheckCXXCompilerFlag)
>  
>  # X265_BUILD must be incremented each time the public API is changed
> -set(X265_BUILD 45)
> +set(X265_BUILD 46)
>  configure_file("${PROJECT_SOURCE_DIR}/x265.def.in"
>                 "${PROJECT_BINARY_DIR}/x265.def")
>  configure_file("${PROJECT_SOURCE_DIR}/x265_config.h.in"
> diff -r 21257215be0f -r d0e4f7ae7d85 source/common/param.cpp
> --- a/source/common/param.cpp	Wed Feb 18 16:20:17 2015 +0530
> +++ b/source/common/param.cpp	Mon Feb 16 14:02:23 2015 +0530
> @@ -571,6 +571,7 @@
>      OPT("repeat-headers") p->bRepeatHeaders = atobool(value);
>      OPT("wpp") p->bEnableWavefront = atobool(value);
>      OPT("ctu") p->maxCUSize = (uint32_t)atoi(value);
> +    OPT("min-cu-size") p->minCUSize = (uint32_t)atoi(value);
>      OPT("tu-intra-depth") p->tuQTMaxIntraDepth = (uint32_t)atoi(value);
>      OPT("tu-inter-depth") p->tuQTMaxInterDepth = (uint32_t)atoi(value);
>      OPT("max-tu-size") p->maxTUSize = (uint32_t)atoi(value);
> @@ -1309,6 +1310,7 @@
>      s += sprintf(s, " bitdepth=%d", p->internalBitDepth);
>      BOOL(p->bEnableWavefront, "wpp");
>      s += sprintf(s, " ctu=%d", p->maxCUSize);
> +    s += sprintf(s, " min-cu-size=%d", p->minCUSize);
>      s += sprintf(s, " max-tu-size=%d", p->maxTUSize);
>      s += sprintf(s, " tu-intra-depth=%d", p->tuQTMaxIntraDepth);
>      s += sprintf(s, " tu-inter-depth=%d", p->tuQTMaxInterDepth);
> diff -r 21257215be0f -r d0e4f7ae7d85 source/x265cli.h
> --- a/source/x265cli.h	Wed Feb 18 16:20:17 2015 +0530
> +++ b/source/x265cli.h	Mon Feb 16 14:02:23 2015 +0530
> @@ -71,6 +71,7 @@
>      { "no-wpp",               no_argument, NULL, 0 },
>      { "wpp",                  no_argument, NULL, 0 },
>      { "ctu",            required_argument, NULL, 's' },
> +    { "min-cu-size",    required_argument, NULL, 's' },

and fixed the 's' short options here. there should be only one

>      { "max-tu-size",    required_argument, NULL, 's' },
>      { "tu-intra-depth", required_argument, NULL, 0 },
>      { "tu-inter-depth", required_argument, NULL, 0 },
> @@ -265,6 +266,7 @@
>      H0("                                 psnr, ssim, grain, zerolatency, fastdecode\n");
>      H0("\nQuad-Tree size and depth:\n");
>      H0("-s/--ctu <64|32|16>              Maximum CU size (WxH). Default %d\n", param->maxCUSize);
> +    H0("   --min-cu-size <64|32|16|8>    Minimum CU size (WxH). Default %d\n", param->minCUSize);
>      H0("   --max-tu-size <32|16|8|4>     Maximum TU size (WxH). Default %d\n", param->maxTUSize);
>      H0("   --tu-intra-depth <integer>    Max TU recursive depth for intra CUs. Default %d\n", param->tuQTMaxIntraDepth);
>      H0("   --tu-inter-depth <integer>    Max TU recursive depth for inter CUs. Default %d\n", param->tuQTMaxInterDepth);
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel

-- 
Steve Borho