[mlpack-git] [mlpack] Bugfix #350 and some fixes in RStarTreeSplit and XTreeSplit (#556)

[Ryan Curtin]

I read through the X tree paper again.  I agree that the DescentType is not specified... the paper just says in Figure 7:

```
follow = choose_subtree(obj);
```

And there is no explanation of what `choose_subtree()` is, so I think it's safe to assume that it's the typical R tree heuristic.  (Maybe it would be possible to use the R* heuristic too, but, I think sticking with the default is good for the X tree typedef.)

Ball trees aren't added to allkrann because of a tricky bug:
https://github.com/mlpack/mlpack/issues/338

---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/556#issuecomment-195605832
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160311/f0f31101/attachment.html>