Skip to content
Will Levine edited this page Jul 23, 2015 · 8 revisions

Remove NMatrix#getrf: This one doesn't make sense, because it's not really analogous to #getrf!, which returns the pivot array, and replaces the original matrix with LU. #getrf just returns the LU part and throws away the pivot array.

Remove NMatrix#lu_decomposition!: This is an alias for #getrf!, but the name seems too similar to #factorize_lu which is a different thing.

Change #getrf!: The clapack_?getrf interface is strange and doesn't match the LAPACK spec. Make #getrf! match LAPACK rather than ATLAS CLAPACK.

Clone this wiki locally