<div dir="ltr">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.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 22, 2016 at 3:14 AM, <span dir="ltr"><<a href="mailto:gopi.satykrishna@multicorewareinc.com" target="_blank">gopi.satykrishna@multicorewareinc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Gopi Satykrishna Akisetty <<a href="mailto:gopi.satykrishna@multicorewareinc.com">gopi.satykrishna@<wbr>multicorewareinc.com</a>><br>
# Date 1474539223 -19800<br>
# Thu Sep 22 15:43:43 2016 +0530<br>
# Node ID 24ecccf7b435e849545e161a0992f4<wbr>52875f6bbc<br>
# Parent bdf098eefd35b70b68419f5a204e60<wbr>f1df4aeae1<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>
______________________________<wbr>_________________<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" rel="noreferrer" target="_blank">https://mailman.videolan.org/<wbr>listinfo/x265-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Deepthi</div>
</div>