<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"><meta name="Generator" content="Microsoft Word 14 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri","sans-serif";}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style></head><body lang="EN-US" link="blue" vlink="purple"><div class="WordSection1"><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">We concluded that it doesn’t make sense to prevent the ability to encode with a fixed GOP size, regardless of whether or not a user wants scenecut enabled.  This patch enables users to specify a min-keyint that is equal to keyint (a fixed keyint) that x265 does not override.  Separately, we’ve also fixed our scenecut option to enable the benefits of scenecut detection with fixed GOP structures.</span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"> </span></p><p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> x265-devel [mailto:<a href="mailto:x265-devel-bounces@videolan.org">x265-devel-bounces@videolan.org</a>] <b>On Behalf Of </b>Deepthi Nandakumar<br><b>Sent:</b> Monday, September 26, 2016 12:01 PM<br><b>To:</b> Development for x265<br><b>Subject:</b> Re: [x265] [PATCH] modify the minimum keyint value</span></p><p class="MsoNormal"> </p><div><p class="MsoNormal">Earlier, keyframeMin was fixed to the range (1, keyframeMax/2 + 1). This patch removes the upper bound, but does not change behaviour wrt scenecut threshold in any way.</p></div><div><p class="MsoNormal"> </p><div><p class="MsoNormal">On Thu, Sep 22, 2016 at 3:14 AM, <<a href="mailto:gopi.satykrishna@multicorewareinc.com" target="_blank">gopi.satykrishna@multicorewareinc.com</a>> wrote:</p><p class="MsoNormal"># HG changeset patch<br># User Gopi Satykrishna Akisetty <<a href="mailto:gopi.satykrishna@multicorewareinc.com">gopi.satykrishna@multicorewareinc.com</a>><br># Date 1474539223 -19800<br>#      Thu Sep 22 15:43:43 2016 +0530<br># Node ID 24ecccf7b435e849545e161a0992f452875f6bbc<br># Parent  bdf098eefd35b70b68419f5a204e60f1df4aeae1<br>modify the minimum keyint value<br><br>This patch enables fixed gop size even when scenecut option is enabled<br><br>diff -r bdf098eefd35 -r 24ecccf7b435 doc/reST/cli.rst<br>--- a/doc/reST/cli.rst  Thu Sep 22 13:26:57 2016 +0530<br>+++ b/doc/reST/cli.rst  Thu Sep 22 15:43:43 2016 +0530<br>@@ -1141,10 +1141,10 @@<br> .. option:: --min-keyint, -i <integer><br><br>        Minimum GOP size. Scenecuts closer together than this are coded as I<br>-       or P, not IDR. Minimum keyint is clamped to be at least half of<br>-       :option:`--keyint`. If you wish to force regular keyframe intervals<br>+       or P, not IDR. If you wish to force regular keyframe intervals<br>        and disable adaptive I frame placement, you must use<br>-       :option:`--no-scenecut`.<br>+       :option:`--no-scenecut` or give minimum keyint equal to keyint<br>+       with scenecut enabled.<br><br>        **Range of values:** >=0 (0: auto)<br><br>diff -r bdf098eefd35 -r 24ecccf7b435 source/encoder/encoder.cpp<br>--- a/source/encoder/encoder.cpp        Thu Sep 22 13:26:57 2016 +0530<br>+++ b/source/encoder/encoder.cpp        Thu Sep 22 15:43:43 2016 +0530<br>@@ -1660,7 +1660,7 @@<br>         double fps = (double)p->fpsNum / p->fpsDenom;<br>         p->keyframeMin = X265_MIN((int)fps, p->keyframeMax / 10);<br>     }<br>-    p->keyframeMin = X265_MAX(1, X265_MIN(p->keyframeMin, p->keyframeMax / 2 + 1));<br>+    p->keyframeMin = X265_MAX(1, p->keyframeMin);<br><br>     if (!p->bframes)<br>         p->bBPyramid = 0;<br>_______________________________________________<br>x265-devel mailing list<br><a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br><a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a></p></div><p class="MsoNormal"><br><br clear="all"></p><div><p class="MsoNormal"> </p></div><p class="MsoNormal">-- </p><div><p class="MsoNormal">Deepthi</p></div></div></div></body></html>