forked from SciRuby/nmatrix
-
Notifications
You must be signed in to change notification settings - Fork 0
API Changes
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. All the callers were confused about this anyway, so I think it's OK to change.