[x265] [PATCH] Support for HLG-graded content and pic_struct

[Vittorio Giovara]

On Thu, Apr 12, 2018 at 4:49 PM, Vittorio Giovara <
vittorio.giovara at gmail.com> wrote:

>
>
>
> On Thu, Apr 12, 2018 at 1:44 PM, <ashok at multicorewareinc.com> wrote:
>
>> # HG changeset patch
>> # User Ashok Kumar Mishra <ashok at multicorewareinc.com>
>> # Date 1523532439 -19800
>> #      Thu Apr 12 16:57:19 2018 +0530
>> # Node ID ace55bce0e3c6e0beb52712fd71e01025bd73a22
>> # Parent  04a337abd70de269cef7d9655365f3a3ebde02aa
>> Support for HLG-graded content and pic_struct
>>
>> diff -r 04a337abd70d -r ace55bce0e3c source/common/param.cpp
>> --- a/source/common/param.cpp   Thu Apr 12 15:10:59 2018 +0530
>> +++ b/source/common/param.cpp   Thu Apr 12 16:57:19 2018 +0530
>> @@ -134,7 +134,8 @@
>>      param->bEmitInfoSEI = 1;
>>      param->bEmitHDRSEI = 0;
>>      param->bEmitIDRRecoverySEI = 0;
>> -    /* CU definitions */
>> +
>> +       /* CU definitions */
>>      param->maxCUSize = 64;
>>      param->minCUSize = 8;
>>      param->tuQTMaxInterDepth = 1;
>> @@ -192,7 +193,8 @@
>>      param->bEnableSAO = 1;
>>      param->bSaoNonDeblocked = 0;
>>      param->bLimitSAO = 0;
>> -    /* Coding Quality */
>> +
>> +       /* Coding Quality */
>>      param->cbQpOffset = 0;
>>      param->crQpOffset = 0;
>>      param->rdPenalty = 0;
>> @@ -302,6 +304,10 @@
>>      param->bLowPassDct = 0;
>>      param->bMVType = 0;
>>      param->bSingleSeiNal = 0;
>> +
>> +       /* SEI messages */
>> +       param->preferredTransferCharacteristics = -1;
>> +       param->pictureStructure = -1;
>>  }
>>
>>  int x265_param_default_preset(x265_param* param, const char* preset,
>> const char* tune)
>> @@ -1030,6 +1036,8 @@
>>          OPT("max-ausize-factor") p->maxAUSizeFactor = atof(value);
>>          OPT("dynamic-refine") p->bDynamicRefine = atobool(value);
>>          OPT("single-sei") p->bSingleSeiNal = atobool(value);
>> +               OPT("atc-sei") p->preferredTransferCharacteristics =
>> atoi(value);
>> +               OPT("pic-struct") p->pictureStructure = atoi(value);
>>          else
>>              return X265_PARAM_BAD_NAME;
>>      }
>> diff -r 04a337abd70d -r ace55bce0e3c source/encoder/encoder.cpp
>> --- a/source/encoder/encoder.cpp        Thu Apr 12 15:10:59 2018 +0530
>> +++ b/source/encoder/encoder.cpp        Thu Apr 12 16:57:19 2018 +0530
>> @@ -338,10 +338,12 @@
>>
>>      if (m_param->bEmitHRDSEI)
>>          m_rateControl->initHRD(m_sps);
>> +
>>      if (!m_rateControl->init(m_sps))
>>          m_aborted = true;
>>      if (!m_lookahead->create())
>>          m_aborted = true;
>> +
>>      initRefIdx();
>>      if (m_param->analysisSave && m_param->bUseAnalysisFile)
>>      {
>>
>
> For the future, could you please keep cosmetic changes (such as the ones
> above) in a separate patch?
> It gets confusing really fast and makes reading history needlessly fast.
>
>
needlessy hard*
-- 
Vittorio
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20180412/20815fe7/attachment-0001.html>