[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