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

[Ryan Curtin]

> +  const StatisticType& Stat() const { return stat; }
> +  //! Return the statistic object for this node.
> +  StatisticType& Stat() { return stat; }
> +
> +  //! Return whether or not this node is a leaf (true if it has no children).
> +  bool IsLeaf() const;
> +
> +  //! Gets the left child of this node.
> +  SpillTree* Left() const { return left; }
> +  //! Modify the left child of this node.
> +  SpillTree*& Left() { return left; }
> +
> +  //! Gets the right child of this node.
> +  SpillTree* Right() const { return right; }
> +  //! Modify the right child of this node.
> +  SpillTree*& Right() { return right; }

If you prefer, I am happy to leave out the `Left()` and `Right()` functions, and have users access the left and right children as `Child(0)` and `Child(1)`.  I think all of the algorithms that use trees should be depending on `Child()` anyway.

---
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/a71b57caa90311f5542180bc0553449c3691395d#r73711046
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160805/cdb04769/attachment-0001.html>