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

[Deepthi Nandakumar]

Here, wouldnt it be better to call this --min-cu-size to go with the
--max-tu-size introduced earlier?

On Mon, Feb 16, 2015 at 4:55 PM, <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 5fdaef859517b16ef37f26ec1a259a5c139744fe
> # Parent  24e3fc60cb4183fff50921a332f0fd0d98cc0f56
> CLI: add new option cu-min(minimum CU size)
>
> diff -r 24e3fc60cb41 -r 5fdaef859517 doc/reST/cli.rst
> --- a/doc/reST/cli.rst  Mon Feb 16 14:33:08 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:: --cu-min, <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 24e3fc60cb41 -r 5fdaef859517 source/CMakeLists.txt
> --- a/source/CMakeLists.txt     Mon Feb 16 14:33:08 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 24e3fc60cb41 -r 5fdaef859517 source/common/param.cpp
> --- a/source/common/param.cpp   Mon Feb 16 14:33:08 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("cu-min") 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);
> @@ -1302,6 +1303,7 @@
>      s += sprintf(s, " bitdepth=%d", p->internalBitDepth);
>      BOOL(p->bEnableWavefront, "wpp");
>      s += sprintf(s, " ctu=%d", p->maxCUSize);
> +    s += sprintf(s, " cu-min=%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 24e3fc60cb41 -r 5fdaef859517 source/x265cli.h
> --- a/source/x265cli.h  Mon Feb 16 14:33:08 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' },
> +    { "cu-min",       required_argument, NULL, 's' },
>      { "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("   --cu-min <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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20150218/eb26ba4f/attachment.html>