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

[Ryan Curtin]

> +   */
> +  static void GetDistinctSamples(arma::uvec& distinctSamples,
> +      const size_t numSamples, const size_t begin, const size_t upperBound);
> +
> +  /**
> +   * Get the median value of the distance from a certain vantage point to a
> +   * number of samples.
> +   *
> +   * @param bound The bound used for this node.
> +   * @param data The dataset used by the binary space tree.
> +   * @param samples The indices of random samples.
> +   * @param vantagePoint The vantage point.
> +   * @param mu The median value.
> +   */
> +  static void GetMedian(const BoundType& bound, const MatType& data,
> +      const arma::uvec& samples,  const size_t vantagePoint, ElemType& mu);

I think here and in `GetSecondMoment()`, it would be better to pass a `MetricType` instead of a `BoundType`.

---
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#r71805911
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160721/ccc76669/attachment.html>