[mlpack-git] [mlpack/mlpack] Spill trees (#747)

[Ryan Curtin]

> + *   booktitle = {Advances in Neural Information Processing Systems 17},
> + *   year = {2005},
> + *   pages = {825--832}
> + * }
> + * @endcode
> + *
> + * This template typedef satisfies the TreeType policy API.
> + *
> + * @see @ref trees, SpillTree, MeanSPTree
> + */
> +template<typename MetricType, typename StatisticType, typename MatType>
> +using SPTree = SpillTree<MetricType,
> +                         StatisticType,
> +                         MatType,
> +                         AxisOrthogonalHyperplane,
> +                         MidpointSpaceSplit>;

I'm not sure if `SPTree` is the best name here.  But the name `SpillTree` is already taken.  Maybe you might consider the same type of thing I did for the cover tree, where the class is called `CoverTree` and the standard template alias is called `StandardCoverTree`?  I'm open to ideas here, and maybe there isn't anything better; it just seems counterintuitive to me to compress `Spill` as `SP`.

-- 
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/747/files/fe090ee13c7cad79e2b7eb8b6690628ba3ead1ed#r74704481
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160814/d5d58c4a/attachment-0001.html>