<p>I think I have gotten myself confused again.  You are right, there is actually no issue with <code>treeNeedsReset</code> since that can only happen for the case of monochromatic search.  So the bug I thought existed, does not exist.</p>

<p>I was not thinking about the overload where the user passes their own query tree; in that case, we have no need to reset the query tree.  I think that we should add documentation ensuring that the user passes a tree with reset bounds, instead of resetting it for them, because (a) that will slow down the <code>Search()</code> method and (b) this is already the "advanced" way to call <code>Search()</code>, so if users want to build their own special tree that possibly has non-reset bounds for their own reasons, we should allow them to do that.</p>

<p>So in short, I agree with your original statement, that we should add documentation, but I got confused and sidetracked while I was reading through trying to understand. :)</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/issues/672#issuecomment-223606789">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFKdh-uA4_e8s77HVO4bfcRx81TbXks5qIESxgaJpZM4Itiy7">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFG2UbFLIccO-GTVhjxsW9khxWNpMks5qIESxgaJpZM4Itiy7.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/issues/672#issuecomment-223606789"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>