[mlpack-git] [mlpack/mlpack] Universal B tree implementation (#746)

[Ryan Curtin]

> @@ -877,7 +879,43 @@ template<typename MetricType,
>           template<typename BoundMetricType, typename...> class BoundType,
>           template<typename SplitBoundType, typename SplitMatType>
>               class SplitType>
> +size_t BinarySpaceTree<MetricType, StatisticType, MatType, BoundType,
> +    SplitType>::PerformSplit(
> +    MatType& data,
> +    const size_t begin,
> +    const size_t count,
> +    const typename UBTreeSplit<BoundType<MetricType>,
> +                               MatType>::SplitInfo& splitInfo)
> +{
> +  return SplitType<BoundType<MetricType>, MatType>::PerformSplit(data, begin,
> +      count, splitInfo);
> +}

Sure, that sounds good to me.  I figured it might be ok in `math` since maybe somewhere else might use it someday, but I agree, maybe we shouldn't move it there until multiple places use it.

-- 
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/746/files/f17843fc8e8ef2b1c4b04d572c521575f20a1f3c#r76270200
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160825/a0f270b2/attachment-0001.html>