<p>In <a href="https://github.com/mlpack/mlpack/pull/742#discussion_r72767969">src/mlpack/methods/preprocess/preprocess_describe_main.cpp</a>:</p>
<pre style='color:#555'>> + double rowStd = arma::stddev(row, population);
> +
> + // Print statistics of the row i.
> + Log::Info << boost::format(numberFormat)
> + % i
> + % arma::var(row, population)
> + % rowMean
> + % rowStd
> + % arma::median(row)
> + % rowMin
> + % rowMax
> + % (rowMax - rowMin) // range
> + % Skewness(row, rowStd, rowMean, population)
> + % Kurtosis(row, rowStd, rowMean, population)
> + % StandardError(row, rowStd) << endl;
> + }
</pre>
<p>We can reduce some duplicate codes</p>
<pre><code>if(CLI::HasParam("dimension")){
printStatResults(dimension);
}else{
for(size_t i = 0; i < data.n_rows; ++i){
printStatResults(i);
}
}
</code></pre>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/742/files/27ac82e876c3d68637d9e72d28bf0dadbc229ace#r72767969">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFNeVbLH6oHWixypfEp2k7DubqmOGks5qacx1gaJpZM4JWax8">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFN_kvFrqCXahTsMuZBKzt2St43eTks5qacx1gaJpZM4JWax8.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/mlpack/mlpack/pull/742/files/27ac82e876c3d68637d9e72d28bf0dadbc229ace#r72767969"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>