<p>I fixed those issues. And I did the refactoring of all RectangleTree classes. Now they should support <code>localDataset</code>. I had to rewrite DiscreteHilbertValue entirely.<br>
But I am not sure of <code>dataset</code>. As stated in <code>src/mlpack/core/tree/example_tree.hpp</code> each tree should satisfy that definition. Particulary <code>size_t Point(const size_t i) const</code> returns the index of a particular point of the node. Maybe we should not remove <code>dataset</code> from the RectangleTree class.</p>

<p>RecursiveHilbertValue is slow since there are a lot of levels of the recursion process. Indeed each double value belongs to [-2^300, 2^300] (roughly). I think we can use RecursiveHilbertValue in order to test DiscreteHilbertValue (they should give equal results).</p>

<p>I have not tested HilbertValueType separately yet. I have some ideas. I'll try them tomorrow.</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-223785541">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFNxxF0qfcb7rVJGH7kw58ZfTHUUFks5qIhTegaJpZM4IrlzT">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFCQFj8Z7yvCrOLOepOkJVjQio3HLks5qIhTegaJpZM4IrlzT.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-223785541"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>