<p>In <a href="https://github.com/mlpack/mlpack/pull/691#discussion_r68334690">src/mlpack/methods/lsh/lsh_search.hpp</a>:</p>
<pre style='color:#555'>> + * projection location, GetAdditionalProbingBins will calculate the T most
> + * likely alternative bin codes (other than queryCode) where a query's
> + * neighbors might be found in.
> + *
> + * @param queryCode vector containing the numProj-dimensional query code.
> + * @param queryCodeNotFloored vector containing the projection location of the
> + * query.
> + * @param T number of additional probing bins.
> + * @param additionalProbingBins matrix. Each column will hold one additional
> + * bin.
> + */
> + void GetAdditionalProbingBins(const arma::vec &queryCode,
> + const arma::vec &queryCodeNotFloored,
> + const size_t T,
> + arma::mat &additionalProbingBins) const;
> +
</pre>
<p>It's a matter of preference, but to be consistent with the rest of the code: when declaring a variable, place the <code>*</code> or <code>&</code> adjacent to the type.</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/e2596c5ac626fe6264d89ec4d546506228e16d56#r68334690">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFIbjDI3UzapaKDPE87tbh6UBXlBaks5qOxq7gaJpZM4IxQW5">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFCymuB3cjf38G5_hMaMMrMythx_Hks5qOxq7gaJpZM4IxQW5.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/e2596c5ac626fe6264d89ec4d546506228e16d56#r68334690"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>