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

[Ryan Curtin]

> +  const std::vector<arma::mat> getProjectionTables() { return projections; }
> +
> +  //! Change the projection tables (Retrains object)
> +  void setProjectionTables(std::vector<arma::mat> projTables)
> +  {
> +    // Simply call Train() with given projection tables
> +    Train(
> +        *referenceSet,
> +        numProj,
> +        numTables,
> +        hashWidth,
> +        secondHashSize,
> +        bucketSize,
> +        projTables
> +        );
> +  };

Same here, we should call it `Projections(vector<mat>&)`.  (Don't forget to make it a reference, otherwise copying is going to happen.  Also I think it should be const.)

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