<div dir="ltr">In finishFrameStats, the pic_out->frameData->sliceType is getting set as I/P/B. For this, we combine the B/BREF from lowres.sliceType and set it as 'B'<div>But the pic_out->sliceType is set to lowres.sliceType which is an enum with 5 values. </div><div>I can see the lowres.sliceTypes able to differentiate between BREF(4) and B(5). </div><div><br></div><div>Can you pls provide your test cli if you are not able to see this in lowres.sliceType. Since we use it in other places as well, this can be a bigger problem if lowres.sliceType </div><div>does not indicate BREF. </div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><font color="#3d85c6" face="garamond, serif">Aarthi Priya Thirumalai</font><div><font size="1" face="garamond, serif" color="#3d85c6">Technical Lead, UHDkit</font></div><div><font size="1" face="garamond, serif" color="#3d85c6">Multicoreware Inc</font></div></div></div></div>
<br><div class="gmail_quote">On Mon, Nov 20, 2017 at 9:12 PM, John Stebbins <span dir="ltr"><<a href="mailto:stebbins@jetheaddev.com" target="_blank">stebbins@jetheaddev.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
<p><tt>BREF is not set in lowres.SliceType. Only B slice type is
getting set in pic_out sliceType. A similar test is already
performed in finishFrameStats because lowres.sliceType does not
distinguish.</tt><br>
</p><div><div class="h5">
<br>
<div class="m_852671179340842076moz-cite-prefix">On 11/19/2017 10:15 PM, Aarthi Priya
Thirumalai wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">
<div class="gmail_extra">
<div>
<div class="m_852671179340842076gmail_signature" data-smartmail="gmail_signature">
<div>I think this change is not necessary since the
sliceType in pic_out is already set from
m_lowres.sliceType. </div>
<div>So, if the B frame is a B_REF, this is reflected from
the lowres.sliceType already.</div>
<div><br>
</div>
<div>Is there any reason, you are explicitly setting it
again?</div>
</div>
</div>
<br>
<div class="gmail_quote">On Fri, Nov 17, 2017 at 4:41 AM, John
Stebbins <span dir="ltr"><<a href="mailto:stebbins@jetheaddev.com" target="_blank">stebbins@jetheaddev.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 John Stebbins <<a href="mailto:jstebbins.hb@gmail.com" target="_blank">jstebbins.hb@gmail.com</a>><br>
# Date 1510861637 28800<br>
# Thu Nov 16 11:47:17 2017 -0800<br>
# Branch stable<br>
# Node ID 5b61fc79bfa2d2bf85f8796001d711<wbr>e5eed1fa1c<br>
# Parent 0b3ba15b33eaed5681f63969702452<wbr>5e9460bb31<br>
encoder: expose BREF slice type in pic_out sliceType<br>
<br>
The AppleTV 4K requires accurate frame dependency
information in the<br>
mp4 sdtp box in order to play 2160p60 content.<br>
<br>
diff -r 0b3ba15b33ea -r 5b61fc79bfa2
source/encoder/encoder.cpp<br>
--- a/source/encoder/encoder.cpp Tue Sep 05
11:21:24 2017 +0530<br>
+++ b/source/encoder/encoder.cpp Thu Nov 16
11:47:17 2017 -0800<br>
@@ -857,6 +857,9 @@<br>
pic_out->pts = outFrame->m_pts;<br>
pic_out->dts = outFrame->m_dts;<br>
pic_out->sliceType =
outFrame->m_lowres.sliceType;<br>
+ if (pic_out->sliceType == X265_TYPE_B
&&<br>
+ IS_REFERENCED(outFrame))<br>
+ pic_out->sliceType =
X265_TYPE_BREF;<br>
pic_out->planes[0] =
recpic->m_picOrg[0];<br>
pic_out->stride[0] =
(int)(recpic->m_stride * sizeof(pixel));<br>
if (m_param->internalCsp !=
X265_CSP_I400)<br>
______________________________<wbr>_________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/l<wbr>istinfo/x265-devel</a><br>
</blockquote>
</div>
<br>
</div>
</div>
<br>
<fieldset class="m_852671179340842076mimeAttachmentHeader"></fieldset>
<br>
<pre>______________________________<wbr>_________________
x265-devel mailing list
<a class="m_852671179340842076moz-txt-link-abbreviated" href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a>
<a class="m_852671179340842076moz-txt-link-freetext" href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/<wbr>listinfo/x265-devel</a>
</pre>
</blockquote>
<br>
</div></div><span class="HOEnZb"><font color="#888888"><pre class="m_852671179340842076moz-signature" cols="120">--
John GnuPG fingerprint: D0EC B3DB C372 D1F1 0B01 83F0 49F1 D7B2 60D4 D0F7
</pre>
</font></span></div>
<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>
<br></blockquote></div><br></div>