[mlpack-git] [mlpack/mlpack] Lsh table access (#663)

[Ryan Curtin]

>     */
>    void Train(const arma::mat& referenceSet,
>               const size_t numProj,
>               const size_t numTables,
>               const double hashWidth = 0.0,
>               const size_t secondHashSize = 99901,
> -             const size_t bucketSize = 500);
> +             const size_t bucketSize = 500,
> +             const std::vector<arma::mat> &projection
> +             = std::vector<arma::mat>()

Do you think we could use `arma::cube` here instead?  We know how many projection tables we have and how many projections are in each table (and it's the same number of projections per table), so we don't need the extra overhead of the vector.  I know that's not code you wrote, but I wanted to ask if you think we could make that change (I'll do it if you think it's reasonable, unless you beat me to it :)).

---
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/663/files/934fe082230f7d8b8f9c3e9c1fe2f9fc25ea493e#r65260027
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160531/41b7ae79/attachment.html>