[mlpack-git] [mlpack] Adds inplace transpose method and some tests. (#408)
Ryan Curtin
notifications at github.com
Mon Feb 23 12:01:02 EST 2015
> @@ -220,7 +220,9 @@ bool Load(const std::string& filename,
>
> // Now transpose the matrix, if necessary.
> if (transpose)
> - matrix = trans(matrix);
> + {
> + arma::inplace_transpose(matrix);
> + }
I don't know where else we might use `inplace_transpose()`. Do you think we should just inline it here instead of adding new files in `arma_extend/`? I don't think this contribution will go back upstream, which is usually what `arma_extend` is for (backporting new functionality, or preparing new stuff to submit upstream).
---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/408/files#r25180471
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20150223/e68ca627/attachment.html>
More information about the mlpack-git
mailing list