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

[Ryan Curtin]

> +  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 );

The same comments apply from earlier; I think this can be shortened and extra spaces removed.

---
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#r68646025
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160627/eb241857/attachment.html>