<p>In <a href="https://github.com/mlpack/mlpack/pull/691#discussion_r68645598">src/mlpack/methods/lsh/lsh_search_impl.hpp</a>:</p>
<pre style='color:#555'>> for (size_t i = 0; i < numTablesToSearch; i++)
> - //allProjInTables.unsafe_col(i) = projections[i].t() * queryPoint;
> - allProjInTables.unsafe_col(i) = projections.slice(i).t() * queryPoint;
> - allProjInTables += offsets.cols(0, numTablesToSearch - 1);
> - allProjInTables /= hashWidth;
> -
> - // Compute the hash value of each key of the query into a bucket of the
> - // 'secondHashTable' using the 'secondHashWeights'.
> - arma::rowvec hashVec = secondHashWeights.t() * arma::floor(allProjInTables);
> + queryCodesNotFloored.unsafe_col(i) = projections.slice(i).t() * queryPoint;
> + queryCodesNotFloored += offsets.cols(0, numTablesToSearch - 1);
> + allProjInTables = arma::floor(queryCodesNotFloored/hashWidth);
</pre>
<p>We should put a space on either side of the division operator <code>/</code>.</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#r68645598">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFEMOoB2vcz-lYOX3QawDrML93iqtks5qQCylgaJpZM4IxQW5">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFGybouBx3ohk7SsEBQp5rwKKVwlJks5qQCylgaJpZM4IxQW5.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#r68645598"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>