<p>Thanks for the contribution.  One of my concerns is that you are using the <code>RTreeDescentHeuristic</code> class instead of the <code>XTreeDescentHeuristic</code> class, which wasn't ever implemented (it was an unfinished GSoC project from 2014).  This is only doable if the X-tree (as specified in the paper) uses the same descent heuristic as the original R tree.  Do you know if that's the case?</p>

<p>Thanks for the RStarTreeSplit fixes too, those must have been oversights when I was refactoring.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/mlpack/mlpack/pull/556#issuecomment-194089749">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFM_Wud_VzVekJrgXl5Z4ojpR_1n6ks5prjaOgaJpZM4HsSai.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/mlpack/mlpack/pull/556#issuecomment-194089749"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>