<p><code>FirstPointIsCentroid</code> is maybe a bit misleading of a name. When I wrote that originally I intended that <code>FirstPointIsCentroid</code> would be true if the first point in the node was the centroid of the region represented by the bound, not the centroid of the descendant points held in the node. So for ball bounds this is the center of the bound. But maybe a better term is <code>FirstPointIsCenter</code>. Either way for the VP tree it should be true. I'll look through the code and see if I can figure out why the traverser is failing.</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/708#issuecomment-231872474">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFI5IWBR0H_FpRUoKTvXrz0vP2lkiks5qUrbTgaJpZM4I_COp">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFB4H4fyZimeqSv_yLTb6ZS_-HgeDks5qUrbTgaJpZM4I_COp.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/708#issuecomment-231872474"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>