<p>In <a href="https://github.com/mlpack/mlpack/pull/523#discussion_r60098393">src/mlpack/core/util/split_data.hpp</a>:</p>
<pre style='color:#555'>> + *TrainTestSplit(input, label, trainData,
> + * testData, trainLabel, testLabel);
> + *@endcode
> + */
> +template<typename T, typename U>
> +void TrainTestSplit(arma::Mat<T> const &input,
> + arma::Row<U> const &inputLabel,
> + arma::Mat<T> &trainData,
> + arma::Mat<T> &testData,
> + arma::Row<U> &trainLabel,
> + arma::Row<U> &testLabel,
> + double const testRatio)
> +{
> + size_t const testSize =
> + static_cast<size_t>(input.n_cols * testRatio);
> + size_t const trainSize = input.n_cols - testSize;
</pre>
<p>I'd agree with this, I'd personally prefer prefix const since it's what's used everywhere.</p>
<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 or <a href="https://github.com/mlpack/mlpack/pull/523/files/c92dd2fc6561b1cf40efb744dcc9233bc3675c91#r60098393">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFEOO3RK23Pq4STl49A2earWfP6feks5p474lgaJpZM4HhWF3.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/523/files/c92dd2fc6561b1cf40efb744dcc9233bc3675c91#r60098393"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>