<div dir="ltr">Signalling each EO class for luma and chroma are fixed 2-bit syntax elements.Signalling BO class luma and chroma are fixed 5-bit syntax elements.<div>As you said signalling offsets depend upon input sequence, shich I have already done. Again signalling sao type(EO or BO) is fixed length syntax elements</div><div>irrespective of input sequence. I have tested with different input sequence and the code also says same thing. I checked the standard and HEVC book. </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 6, 2016 at 9:16 PM, chen <span dir="ltr"><<a href="mailto:chenm003@163.com" target="_blank">chenm003@163.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><pre><span class="">>-        m_entropyCoder.load(m_rdContexts.temp);
>-        m_entropyCoder.resetBits();
>-
>-        for (int compIdx = 0; compIdx < 2; compIdx++)
>-            m_entropyCoder.codeSaoOffsetEO(m_offset[compIdx + 1][typeIdx] + 1, typeIdx, compIdx + 1);
>-
>-        uint32_t estRate = m_entropyCoder.getNumberOfWrittenBits();
>-        int64_t cost = calcSaoRdoCost((estDist[0] + estDist[1]), estRate, lambda[1]);
>+        // sao_eo_class_luma and sao_eo_class_chroma are fixed-length 2-bit syntax elements.
>+        cost += eo_type_cost + calcSaoRdoCost(0, 2, </span>lambda[1]);
> 
it is not fixed-length, the <span style="line-height:23.8px">codeSaoOffsetEO output depends on Offset and plane_idx, it is different in difference input sequence</span></pre></div><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" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
<br></blockquote></div><br></div>