<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 24, 2013 at 12:19 AM, Steve Borho <span dir="ltr"><<a href="mailto:steve@borho.org" target="_blank">steve@borho.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Wed, Oct 23, 2013 at 4:20 AM,  <span dir="ltr"><<a href="mailto:shazeb@multicorewareinc.com" target="_blank">shazeb@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 Shazeb Nawaz Khan <<a href="mailto:shazeb@multicorewareinc.com" target="_blank">shazeb@multicorewareinc.com</a>><br>
# Date 1382520034 -19800<br>
#      Wed Oct 23 14:50:34 2013 +0530<br>
# Node ID 98e84472bb7ae8aa60bdcffb5972b4d20017ab79<br>
# Parent  6d96d64c4e9a2c526b57274760a7147241328cb3<br>
Displaying the number of weighted P frames used in console log<br>
<br>
diff -r 6d96d64c4e9a -r 98e84472bb7a source/Lib/TLibCommon/TComSlice.cpp<br>
--- a/source/Lib/TLibCommon/TComSlice.cpp       Tue Oct 22 23:36:36 2013 +0530<br>
+++ b/source/Lib/TLibCommon/TComSlice.cpp       Wed Oct 23 14:50:34 2013 +0530<br>
@@ -125,6 +125,7 @@<br>
     m_enableTMVPFlag = true;<br>
     m_ssim = 0;<br>
     m_ssimCnt = 0;<br>
+    m_numWPRefs = 0;<br>
 }<br>
<br>
 bool TComSlice::getRapPicFlag()<br>
diff -r 6d96d64c4e9a -r 98e84472bb7a source/Lib/TLibCommon/TComSlice.h<br>
--- a/source/Lib/TLibCommon/TComSlice.h Tue Oct 22 23:36:36 2013 +0530<br>
+++ b/source/Lib/TLibCommon/TComSlice.h Wed Oct 23 14:50:34 2013 +0530<br>
@@ -1375,8 +1375,9 @@<br>
<br>
 public:<br>
<br>
-    MotionReference * m_mref[2][MAX_NUM_REF + 1];<br>
-    wpScalingParam  m_weightPredTable[2][MAX_NUM_REF][3]; // [REF_PIC_LIST_0 or REF_PIC_LIST_1][refIdx][0:Y, 1:U, 2:V]<br>
+    MotionReference  *m_mref[2][MAX_NUM_REF + 1];<br>
+    wpScalingParam    m_weightPredTable[2][MAX_NUM_REF][3]; // [REF_PIC_LIST_0 or REF_PIC_LIST_1][refIdx][0:Y, 1:U, 2:V]<br>
+    int               m_numWPRefs;                          // number of references for which unidirectional weighted prediction is used<br>
<br>
     /* SSIM values per frame */<br>
     double          m_ssim;<br>
diff -r 6d96d64c4e9a -r 98e84472bb7a source/encoder/encoder.cpp<br>
--- a/source/encoder/encoder.cpp        Tue Oct 22 23:36:36 2013 +0530<br>
+++ b/source/encoder/encoder.cpp        Wed Oct 23 14:50:34 2013 +0530<br>
@@ -94,6 +94,7 @@<br>
     m_lookahead = new Lookahead(this, m_threadPool);<br>
     m_dpb = new DPB(this);<br>
     m_rateControl = new RateControl(this);<br>
+    m_numWPFrames = 0;<br>
 }<br>
<br>
 void Encoder::destroy()<br>
@@ -249,6 +250,8 @@<br>
             pic_out->stride[2] = recpic->getCStride();<br>
         }<br>
<br>
+        if (out->getSlice()->m_numWPRefs > 0)<br>
+            m_numWPFrames++;<br></blockquote></div></div></div></div></div></blockquote><div><br></div><div>..........(1)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
         uint64_t bits = calculateHashAndPSNR(out, nalunits);<br>
         // Allow this frame to be recycled if no frame encoders are using it for reference<br>
         ATOMIC_DEC(&out->m_countRefEncoders);<br>
@@ -293,6 +296,12 @@<br>
         m_analyzeB.printOut('b', fps);<br>
         m_analyzeAll.printOut('a', fps);<br>
     }<br>
+<br>
+    if (param.bEnableWeightedPred)<br>
+    {<br>
+        fprintf(stderr, "x265 [info]: %d weighted P frames used\n", m_numWPFrames);<br></blockquote><div><br></div></div></div><div>this should use x265_log().  Perhaps it should be something like:  "%d of %d (%d%%) P frames weighted"</div>
</div></div></div></blockquote><div><br></div><div>ok, What do we want to print with the third %d? total number of frames encoded?</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+    }<br>
+<br>
<br>
 #if _SUMMARY_OUT_<br>
     m_analyzeAll.printSummaryOut(fps);<br>
@@ -326,6 +335,7 @@<br>
         stats->globalSsim = 0;<br>
         stats->globalPsnr = 0;<br>
     }<br>
+    stats->totalWPFrames = m_numWPFrames;<br>
 }<br>
<br>
 #define VERBOSE_RATE 0<br>
diff -r 6d96d64c4e9a -r 98e84472bb7a source/encoder/encoder.h<br>
--- a/source/encoder/encoder.h  Tue Oct 22 23:36:36 2013 +0530<br>
+++ b/source/encoder/encoder.h  Wed Oct 23 14:50:34 2013 +0530<br>
@@ -69,6 +69,9 @@<br>
     // quality control<br>
     TComScalingList    m_scalingList;      ///< quantization matrix information<br>
<br>
+    // weighted prediction<br>
+    int                m_numWPFrames;      // number of Unidirectional weighted frames used<br>
+<br>
 public:<br>
<br>
     x265_nal_t *m_nals;<br>
diff -r 6d96d64c4e9a -r 98e84472bb7a source/encoder/frameencoder.cpp<br>
--- a/source/encoder/frameencoder.cpp   Tue Oct 22 23:36:36 2013 +0530<br>
+++ b/source/encoder/frameencoder.cpp   Wed Oct 23 14:50:34 2013 +0530<br>
@@ -426,7 +426,10 @@<br>
         {<br>
             TComPicYuv *recon = slice->getRefPic(list, ref)->getPicYuvRec();<br>
             if ((slice->isInterP() && slice->getPPS()->getUseWP()))<br>
+            {<br>
                 w = slice->m_weightPredTable[list][ref];<br>
+                slice->m_numWPRefs++;<br></blockquote><div><br></div></div></div><div>this is counting number of weighted references, which isn't quite the same thing.  The increment needs to be outside this loop</div>
</div></div></div></blockquote><div><br></div><div>this is just collecting more information for future convenience; please check (1) above, encoder gets the sought number</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
+            }<br>
             slice->m_mref[list][ref] = recon->generateMotionReference(w);<br>
         }<br>
     }<br>
diff -r 6d96d64c4e9a -r 98e84472bb7a source/x265.h<br>
--- a/source/x265.h     Tue Oct 22 23:36:36 2013 +0530<br>
+++ b/source/x265.h     Wed Oct 23 14:50:34 2013 +0530<br>
@@ -241,6 +241,7 @@<br>
     double    globalSsim;<br>
     double    accBits;<br>
     uint32_t  totalNumPics;<br>
+    uint32_t  totalWPFrames;    /* Total number of Uni-directional weighted frames used */<br>
 }<br>
 x265_stats_t;<br>
<br></div>
_______________________________________________<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" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br>Steve Borho
</font></span></div></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" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
<br></blockquote></div><br></div></div>