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

[Santhoshini Sekar]

yeah, that would be better. I'll send out a patch.

On Wed, Feb 18, 2015 at 4:50 PM, Deepthi Nandakumar <
deepthi at multicorewareinc.com> wrote:

> 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
>>
>
>
> _______________________________________________
> 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/2212162c/attachment-0001.html>