<p>In <a href="https://github.com/mlpack/mlpack/pull/691#discussion_r68646025">src/mlpack/methods/lsh/lsh_search_impl.hpp</a>:</p>
<pre style='color:#555'>> + if (T > 0)
> + {
> + for (size_t i = 0; i < numTablesToSearch; ++i)
> + {
> + // Construct this table's probing sequence of length T.
> + arma::mat additionalProbingBins;
> + GetAdditionalProbingBins(allProjInTables.unsafe_col(i),
> + queryCodesNotFloored.unsafe_col(i),
> + T,
> + additionalProbingBins);
> +
> + // Map each probing bin to a bin in secondHashTable (just like we did for
> + // the primary hash table).
> + hashMat(arma::span(1, T), i) = // Compute code of rows 1:end of column i
> + arma::conv_to< arma::Col<size_t> >:: // floor by typecasting to size_t
> + from( secondHashWeights.t() * additionalProbingBins );
</pre>
<p>The same comments apply from earlier; I think this can be shortened and extra spaces removed.</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/691/files/71eda99e207a9b91581504e11a9e14d38838bc9f#r68646025">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFFeP4JULosFPIpeLNk2SN9ow3G35ks5qQC1SgaJpZM4IxQW5">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFLAP-yUSDTA18HQrAQp3WV_eFK1Zks5qQC1SgaJpZM4IxQW5.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/691/files/71eda99e207a9b91581504e11a9e14d38838bc9f#r68646025"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>