<p>In <a href="https://github.com/mlpack/mlpack/pull/726#discussion_r73706845">src/mlpack/core/tree/binary_space_tree/rp_tree_max_split_impl.hpp</a>:</p>
<pre style='color:#555'>&gt; +    arma::uvec&amp; distinctSamples,
&gt; +    const size_t begin,
&gt; +    const size_t count,
&gt; +    const size_t numSamples)
&gt; +{
&gt; +  arma::Col&lt;size_t&gt; samples;
&gt; +
&gt; +  samples.zeros(count);
&gt; +
&gt; +  for (size_t i = 0; i &lt; numSamples; i++)
&gt; +    samples [ (size_t) math::RandInt(count) ]++;
&gt; +
&gt; +  distinctSamples = arma::find(samples &gt; 0);
&gt; +
&gt; +  distinctSamples += begin;
&gt; +}
</pre>
<p>Ah, right, that is part of <a href="https://github.com/mlpack/mlpack/pull/708" class="issue-link js-issue-link" data-url="https://github.com/mlpack/mlpack/issues/708" data-id="162438464" data-error-text="Failed to load issue title" data-permission-text="Issue title is private">#708</a>.  I left a comment there recently asking if you think it is ready to merge for now, so if you agree it's ready for now then I'll go ahead and merge it, and you can refactor the code here.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<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/726/files/e14a26f3e209af0608cb61ecaa5e30f253e72e9b#r73706845">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFFyBjikybVnrYwuMkr4MDkgs9mp8ks5qc1C5gaJpZM4JOuGE">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFDAFloTgIn7I1hFE4KbUs-YFERhnks5qc1C5gaJpZM4JOuGE.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/726/files/e14a26f3e209af0608cb61ecaa5e30f253e72e9b#r73706845"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/mlpack/mlpack","title":"mlpack/mlpack","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/modules/aws/aws-bg.jpg","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/mlpack/mlpack"}},"updates":{"snippets":[{"icon":"PERSON","message":"@rcurtin in #726: Ah, right, that is part of #708.  I left a comment there recently asking if you think it is ready to merge for now, so if you agree it's ready for now then I'll go ahead and merge it, and you can refactor the code here."}],"action":{"name":"View Pull Request","url":"https://github.com/mlpack/mlpack/pull/726/files/e14a26f3e209af0608cb61ecaa5e30f253e72e9b#r73706845"}}}</script>