<p>In <a href="https://github.com/mlpack/mlpack/pull/664#discussion_r67892689">src/mlpack/core/tree/rectangle_tree/hilbert_r_tree_split.hpp</a>:</p>
<pre style='color:#555'>> + * Defintion of the HilbertRTreeSplit class, a class that splits the nodes of an R
> + * tree, starting at a leaf node and moving upwards if necessary.
> + */
> +#ifndef MLPACK_CORE_TREE_RECTANGLE_TREE_HILBERT_R_TREE_SPLIT_HPP
> +#define MLPACK_CORE_TREE_RECTANGLE_TREE_HILBERT_R_TREE_SPLIT_HPP
> +
> +#include <mlpack/core.hpp>
> +
> +namespace mlpack {
> +namespace tree /** Trees and tree-building procedures. */ {
> +
> +/**
> + * The order of the splitting policy. The Hilbert R tree splits a node
> + * on overflow, turnung splitOrder node to (splitOrder+1) nodes.
> + */
> +constexpr int splitOrder = 2;
</pre>
<p>I don't know if it's good to just leave <code>splitOrder</code> open in the <code>mlpack::tree</code> namespace, maybe we should put it inside <code>HilbertRTreeSplit</code> as a static member---or alternately make it a template parameter. Whatever you want to do is up to you, I'd just like to get it out of <code>mlpack::tree</code> since there is a lot of other stuff in that namespace too.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<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/files/f9127cea62d5ce1ad8d5f59931de108a2f7cdd9b#r67892689">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFBTMMvL8zsG2iyYrlOjRkCDytzAuks5qOASogaJpZM4IrlzT">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFNLWKWx8wIZw-E5degrePZkfGsslks5qOASogaJpZM4IrlzT.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/files/f9127cea62d5ce1ad8d5f59931de108a2f7cdd9b#r67892689"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>