<p>In <a href="https://github.com/mlpack/mlpack/pull/523#discussion_r60094845">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>Sorry to be picky, but maybe we can agree on a style? Do we use const as suffix or prefix, or are we going to accept both styles?</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#r60094845">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFMYWFq7-0EfAhit1k7lXpXMUh6FJks5p47kSgaJpZM4HhWF3.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#r60094845"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>