[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:
-------------- 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