[mlpack-git] [mlpack/mlpack] Fixed a segfault in RPlusTreeSplit. Fixed traits for the R+/R++ tree. (#724)

[Ryan Curtin]

> +         template<typename> class SweepType,
> +         typename DescentType,
> +         template<typename> class AuxiliaryInformationType>
> +class TreeTraits<RectangleTree<MetricType,
> +    StatisticType,
> +    MatType,
> +    RPlusTreeSplit<SplitPolicyType,
> +                   SweepType>,
> +    DescentType,
> +    AuxiliaryInformationType>>
> +{
> + public:
> +  /**
> +   * The R+/R++ tree can't have overlapping children.
> +   */
> +  static const bool HasOverlappingChildren = false;

I thought that there was a degenerate case that could occur with the R+ tree where nodes could end up overlapping.  I don't remember the exact details; is that still true?  Or did your code work around that situation?

---
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/724/files/2ec9b6ca75c8987e5bae064aac3608b4addcf235#r71200103
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160718/1152900c/attachment-0001.html>