<blockquote>
<p>I am pretty sure your code had a subtle bug where the user could initialize the LSHSearch object without training, but then the destructor would still try to delete the std::vector* object which would cause a crash<br>
You're right I didn't even think about that case.</p>
</blockquote>

<p>I like this idea, I think it's quite an improvement over the current implementation. I'm still not clear on how we can first use secondHashSize as the hash size and then compress the empty buckets though. I'll look into your and Pari's code to see that.</p>

<p>I think I can have this working in one or two days... Though I have some final stuff to do with my thesis and I might not be very active till Wednesday, but I'll find time to work on this. So if there isn't a very big hurry, wait for me and release once this is done. I'll do my best to deliver it as soon as possible :)</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/675#issuecomment-223829360">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFC7caYV1MTePLCbNrRC_8zrexHsKks5qIxdrgaJpZM4IuAcT">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFFdPekjxfKvntRyEBB9clBcbzO6Cks5qIxdrgaJpZM4IuAcT.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/675#issuecomment-223829360"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>