<p>In <a href="https://github.com/mlpack/mlpack/pull/699#discussion_r69022655">src/mlpack/core/tree/rectangle_tree/r_plus_tree_split.hpp</a>:</p>
<pre style='color:#555'>> + * tree, starting at a leaf node and moving upwards if necessary.
> + */
> +#ifndef MLPACK_CORE_TREE_RECTANGLE_TREE_R_PLUS_TREE_SPLIT_HPP
> +#define MLPACK_CORE_TREE_RECTANGLE_TREE_R_PLUS_TREE_SPLIT_HPP
> +
> +#include <mlpack/core.hpp>
> +
> +namespace mlpack {
> +namespace tree /** Trees and tree-building procedures. */ {
> +
> +/**
> + * The RPlusTreeSplit class performs the split process of a node on overflow.
> + *
> + * @tparam SplitPolicyType The class that helps to determine the subtree into
> + * which we should insert a child node.
> + * @tparam SweepType The class that sweeps a node along an axis.
</pre>
<p>Can you document more what the SweepType class needs to do? I understand the classes you have written now that I have read it, but the documentation here doesn't give very much insight to someone who is just skimming the code.</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/699/files/e165d759f9ae612b9965f70fbbf8abdb19dc8d07#r69022655">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFMnL2T8NxLDnJ91Xt4hCXfr9tOWtks5qQtebgaJpZM4I3ATw">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFGBAOE5mPb9DyY3YTI8At0iax0nOks5qQtebgaJpZM4I3ATw.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/699/files/e165d759f9ae612b9965f70fbbf8abdb19dc8d07#r69022655"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>