<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 13, 2017 at 7:09 PM, <span dir="ltr"><<a href="mailto:praveen@multicorewareinc.com" target="_blank">praveen@multicorewareinc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"># HG changeset patch<br>
# User Praveen Tiwari <<a href="mailto:praveen@multicorewareinc.com">praveen@multicorewareinc.com</a>><br>
# Date 1510561483 -19800<br>
# Mon Nov 13 13:54:43 2017 +0530<br>
# Node ID 02f21c4eafe13b52966f3fc1f925bb<wbr>34070d647d<br>
# Parent 65eff30eb192d836b52edb5c3e2e6d<wbr>eae06dabf7<br>
analysis: update analysisReuseLevel 7 for analysis sharing<br></blockquote><div><br></div><div>According to the docs (<a href="http://x265.readthedocs.io/en/default/cli.html#cmdoption-analysis-reuse-level">http://x265.readthedocs.io/en/default/cli.html#cmdoption-analysis-reuse-level</a>), analysis-reuse-level 5-9 are currently different from 10. Your patch here seems inconsistent with that documented behavior.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
diff -r 65eff30eb192 -r 02f21c4eafe1 source/encoder/analysis.cpp<br>
--- a/source/encoder/analysis.cpp Mon Nov 13 12:20:50 2017 +0530<br>
+++ b/source/encoder/analysis.cpp Mon Nov 13 13:54:43 2017 +0530<br>
@@ -2301,7 +2301,7 @@<br>
for (uint32_t part = 0; part < numPU; part++)<br>
{<br>
PredictionUnit pu(<a href="http://mode.cu" rel="noreferrer" target="_blank">mode.cu</a>, cuGeom, part);<br>
- if (m_param->analysisReuseLevel == 10)<br>
+ if (m_param->analysisReuseLevel >= 7)<br>
{<br>
analysis_inter_data* interDataCTU = (analysis_inter_data*)m_frame-<wbr>>m_analysisData.interData;<br>
int cuIdx = (mode.cu.m_cuAddr * parentCTU.m_numPartitions) + cuGeom.absPartIdx;<br>
@@ -2407,7 +2407,7 @@<br>
if (m_slice->m_pps->bUseDQP && nextDepth <= m_slice->m_pps->maxCuDQPDepth)<br>
nextQP = setLambdaFromQP(parentCTU, calculateQpforCuSize(<wbr>parentCTU, childGeom));<br>
<br>
- int lamdaQP = m_param->analysisReuseLevel == 10 ? nextQP : lqp;<br>
+ int lamdaQP = m_param->analysisReuseLevel >= 7 ? nextQP : lqp;<br>
<br>
if (split)<br>
m_param->rdLevel > 4 ? compressInterCU_rd5_6(<wbr>parentCTU, childGeom, nextQP) : compressInterCU_rd0_4(<wbr>parentCTU, childGeom, nextQP);<br>
diff -r 65eff30eb192 -r 02f21c4eafe1 source/encoder/encoder.cpp<br>
--- a/source/encoder/encoder.cpp Mon Nov 13 12:20:50 2017 +0530<br>
+++ b/source/encoder/encoder.cpp Mon Nov 13 13:54:43 2017 +0530<br>
@@ -3038,7 +3038,7 @@<br>
CHECKED_MALLOC(interData-><wbr>mergeFlag, uint8_t, analysis->numPartitions * analysis->numCUsInFrame);<br>
}<br>
<br>
- if (m_param->analysisReuseLevel == 10)<br>
+ if (m_param->analysisReuseLevel >= 7)<br>
{<br>
CHECKED_MALLOC(interData-><wbr>interDir, uint8_t, analysis->numPartitions * analysis->numCUsInFrame);<br>
for (int dir = 0; dir < numDir; dir++)<br>
@@ -3115,7 +3115,7 @@<br>
X265_FREE(((analysis_inter_<wbr>data*)analysis->interData)-><wbr>mergeFlag);<br>
X265_FREE(((analysis_inter_<wbr>data*)analysis->interData)-><wbr>partSize);<br>
}<br>
- if (m_param->analysisReuseLevel == 10)<br>
+ if (m_param->analysisReuseLevel >= 7)<br>
{<br>
X265_FREE(((analysis_inter_<wbr>data*)analysis->interData)-><wbr>interDir);<br>
int numDir = analysis->sliceType == X265_TYPE_P ? 1 : 2;<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></div></div>