[mlpack-git] [mlpack/mlpack] Implementation of Multiprobe LSH (#691)

[Ryan Curtin]

> +
> +    // If dimesnion is unseen thus far, mark it as seen.
> +    if ( check[i % numProj] == false )
> +      check[i % numProj] = true;
> +    else
> +      return false; // If dimension was seen before, set is not valid.
> +  }
> +
> +  // If we didn't fail, set is valid.
> +  return true;
> +}
> +
> +// Compute additional probing bins for a query
> +template<typename SortPolicy>
> +void LSHSearch<SortPolicy>::GetAdditionalProbingBins(
> +                            const arma::vec& queryCode,

The spacing seems odd here; if we can't fit the arguments on the same line as the function name, then we should just indent them four spaces from the start of the line.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/691/files/71eda99e207a9b91581504e11a9e14d38838bc9f#r68644470
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160627/21527f6b/attachment-0001.html>