[mlpack-git] [mlpack/mlpack] Vantage point tree (#708)

[Ryan Curtin]

> +   *    after the rearrangement.
> +   * @param oldFromNew Vector which will be filled with the old positions for
> +   *    each new point.
> +   */
> +  static bool SplitNode(const BoundType& bound,
> +                        MatType& data,
> +                        const size_t begin,
> +                        const size_t count,
> +                        size_t& splitCol,
> +                        std::vector<size_t>& oldFromNew);
> + private:
> +  /**
> +   * The maximum number of samples used for vantage point estimation and for
> +   * estimation of the median.
> +   */
> +  static const size_t maxNumSamples = 100;

It might make sense to have this be a template parameter defaulting to 100, but I don't have a strong opinion either way.

---
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/708/files/300882ac96e7a663e3e303ca0c45c14c6fafe1a6#r71805186
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160721/ef5a8663/attachment.html>