<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 27, 2013 at 11:52 PM, Aarthi Thirumalai <span dir="ltr"><<a href="mailto:aarthi@multicorewareinc.com" target="_blank">aarthi@multicorewareinc.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"># HG changeset patch<br>
# User Aarthi Thirumalai<br>
# Date 1380342254 -19800<br>
#      Sat Sep 28 09:54:14 2013 +0530<br>
# Node ID 0fa415d022c56451e6580c5b2bee793498d986c8<br>
# Parent  4014edcf215747ba4ac8147b1168f8edc6f5d64c<br>
Add CLI options for turning on PSNR, SSIM metrics.<br>
By default , PSNR is turned on , SSIM is off.<br></blockquote><div><br></div><div>I've changed this to:</div><div><br></div><div><div>cli: add options for toggling PSNR and SSIM metrics</div><div><br></div><div>By default (for now) PSNR is turned on, SSIM is off.</div>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
diff -r 4014edcf2157 -r 0fa415d022c5 source/common/common.cpp<br>
--- a/source/common/common.cpp  Fri Sep 27 02:18:36 2013 -0500<br>
+++ b/source/common/common.cpp  Sat Sep 28 09:54:14 2013 +0530<br>
@@ -178,6 +178,10 @@<br>
     param->rc.qpStep = 4;<br>
     param->rc.rateControlMode = X265_RC_CQP;<br>
     param->rc.qp = 32;<br>
+<br>
+    /* Quality Measurement Metrics */<br>
+    param->bEnablePsnr = 1;<br>
+    param->bEnableSsim = 0;<br>
 }<br>
<br>
 extern "C"<br>
@@ -428,6 +432,8 @@<br>
     }<br>
     TOOLOPT(param->bEnableWeightedPred, "weightp");<br>
     TOOLOPT(param->bEnableWeightedBiPred, "weightbp");<br>
+    TOOLOPT(param->bEnableSsim, "ssim");<br>
+    TOOLOPT(param->bEnablePsnr, "psnr");<br></blockquote><div><br></div><div>these aren't coding tools, and it will be apparent from the log which are enabled. dropping these</div><div><br></div><div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
     fprintf(stderr, "\n");<br>
     fflush(stderr);<br>
 }<br>
diff -r 4014edcf2157 -r 0fa415d022c5 source/x265.h<br>
--- a/source/x265.h     Fri Sep 27 02:18:36 2013 -0500<br>
+++ b/source/x265.h     Sat Sep 28 09:54:14 2013 +0530<br>
@@ -266,6 +266,9 @@<br>
     // debugging<br>
     int       decodedPictureHashSEI;           ///< Checksum(3)/CRC(2)/MD5(1)/disable(0) acting on decoded picture hash SEI message<br>
<br>
+    //Quality Measurement Metrics<br>
+    int        bEnablePsnr;<br>
+    int        bEnableSsim;<br></blockquote><div><br></div><div>white-space cleaned up, changed comment to "// quality metrics" to match comments above</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

     struct<br>
     {<br>
         int       bitrate;<br>
diff -r 4014edcf2157 -r 0fa415d022c5 source/x265opts.h<br>
--- a/source/x265opts.h Fri Sep 27 02:18:36 2013 -0500<br>
+++ b/source/x265opts.h Sat Sep 28 09:54:14 2013 +0530<br>
@@ -96,3 +96,7 @@<br>
<br>
 HELP("SEI options:")<br>
 OPT("hash",            param->decodedPictureHashSEI,  required_argument, 0, "Decoded Picture Hash SEI 0: disabled, 1: MD5, 2: CRC, 3: Checksum ")<br>
+<br>
+HELP("Quality Measurement metrices: ")<br>
+OPT("ssim",            param->bEnableSsim,              required_argument, 0, "Log SSIM metric 0:disabled (defaullt), 1: enabled")<br>
+OPT("psnr",            param->bEnablePsnr,              required_argument, 0, "Log PSNR metric 0:disabled, 1: enabled(default)")<br></blockquote><div><br></div><div>boolean options are handled differently.  I changed these to:</div>
<div><br></div><div><div>+HELP("Quality reporting metrics: ")</div><div>+OPT("no-ssim",         param->bEnableSsim,                  no_argument, 0, "Disable reporting SSIM metric scores")</div>
<div>+OPT("ssim",            param->bEnableSsim,                  no_argument, 0, "Enable reporting SSIM metric scores")</div><div>+OPT("no-psnr",         param->bEnablePsnr,                  no_argument, 0, "Disable reporting PSNR metric scores")</div>
<div>+OPT("psnr",            param->bEnablePsnr,                  no_argument, 0, "Enable reporting PSNR metric scores")</div></div><div><br></div><div>and moved them above the debugging hash option.  Note that it's not required to document defaults in these help strings because they are reported separately.</div>
<div><br></div><div>--</div><div>Steve Borho</div></div></div></div>