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

[Ryan Curtin]

> +      if ( minscore2 > scores[s] && s != minloc) //second smallest
> +      {
> +        minscore2 = scores[s];
> +        minloc2 = s;
> +      }
> +    }
> +
> +    // Add or subtract 1 to create second-lowest scoring vector.
> +    additionalProbingBins(positions[minloc2], 1) += actions[minloc2];
> +    return;
> +  }
> +
> +  // General case: more than 2 perturbation vectors require use of minheap.
> +
> +  // Sort everything in increasing order.
> +  arma::Col<long long unsigned int> sortidx = arma::sort_index(scores);

I think this was mentioned somewhere else, so forgive any redundancy, but probably `arma::uword` should be used here instead of `long long unsigned int`.  You can just use `arma::uvec` since that is a typedef of `arma::Col<arma::uword>`.

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