<p>Okay, I have a couple other comments; if you can address these and the ones in the diffs, then if you are happy with the code I will merge it.</p>

<ul>
<li><p>Should we remove <code>RecursiveHilbertValue</code>?  All of the timing tests I am doing indicate that it takes multiple orders of magnitude longer to construct a tree that way.</p></li>
<li><p>The speed of the Hilbert R tree for searching seems comparable to (sometimes better, sometimes worse) than the R* tree.  I think that is about the expected performance.</p></li>
</ul>

<p>Sorry for the delay on my end with the review here... this was a lot of code to look through. :)</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/664#issuecomment-227545526">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFA6Aah7l6MP3vqoHcmtZ-onZTUgeks5qODsOgaJpZM4IrlzT">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFFBa54N9eWaD27dtBj_xGNfrXK6qks5qODsOgaJpZM4IrlzT.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/664#issuecomment-227545526"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>