My concern is that the candidate size counting would take more time in<br>
multiprobe. On the other hand that should not be a big factor in total time.<br>
<br>
At some point I was confused and thought multiprobe would also return<br>
larger candidate sets, but that&#39;s not true - if run correctly it should<br>
return smaller sets since there&#39;s more hash buckets but fewer tables. So I<br>
think unique would actually be the way to go here.<br>
<br>
I think you should go ahead and merge it, and I&#39;ll make the corresponding<br>
changes in the multiprobe code. As we discussed the hybrid option might not<br>
be so good - if I remember well we did shave off a bit of time but the code<br>
will become too complex once multiprobe is added.<br>
<br>
On Sun, Jun 5, 2016 at 9:34 PM, Ryan Curtin &lt;notifications@github.com&gt;<br>
wrote:<br>
<br>
&gt; What did you want to do with this PR? I am happy to merge it in now, but I<br>
&gt; remember that you said you thought it would be less effective for<br>
&gt; multiprobe LSH. (Maybe the thing to do for multiprobe LSH is just to factor<br>
&gt; in however many extra tables are being checked into the calculation?)<br>
&gt;<br>
&gt; —<br>
&gt; You are receiving this because you authored the thread.<br>
&gt; Reply to this email directly, view it on GitHub<br>
&gt; &lt;https://github.com/mlpack/mlpack/pull/623#issuecomment-223829195&gt;, or mute<br>
&gt; the thread<br>
&gt; &lt;https://github.com/notifications/unsubscribe/AFxX3Nw55hdLVdG4NB9rmPyWP_CRaSNbks5qIxawgaJpZM4II_F4&gt;<br>
&gt; .<br>
&gt;<br>


<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/623#issuecomment-223830325">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFMbLgQp3rgTrX_fvs2GxOFZOzm5Gks5qIxuCgaJpZM4II_F4">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFBRZXK6QCJQHs33NM9bfZzCZ3dH9ks5qIxuCgaJpZM4II_F4.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/623#issuecomment-223830325"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>