[mlpack-git] [mlpack/mlpack] Random projection trees (#726)

[Ryan Curtin]

> +    arma::uvec& distinctSamples,
> +    const size_t begin,
> +    const size_t count,
> +    const size_t numSamples)
> +{
> +  arma::Col<size_t> samples;
> +
> +  samples.zeros(count);
> +
> +  for (size_t i = 0; i < numSamples; i++)
> +    samples [ (size_t) math::RandInt(count) ]++;
> +
> +  distinctSamples = arma::find(samples > 0);
> +
> +  distinctSamples += begin;
> +}

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.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/726/files/e14a26f3e209af0608cb61ecaa5e30f253e72e9b#r73706845
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160805/99a0f27d/attachment.html>