Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lufact should not require a square matrix #10970

Closed
gwhowell opened this issue Apr 23, 2015 · 0 comments
Closed

lufact should not require a square matrix #10970

gwhowell opened this issue Apr 23, 2015 · 0 comments
Labels
domain:arrays:sparse Sparse arrays domain:linear algebra Linear algebra status:help wanted Indicates that a maintainer wants help on an issue or pull request

Comments

@gwhowell
Copy link

Here's the error code

function lufact{Tv<:UMFVTypes,Ti<:UMFITypes}(S::SparseMatrixCSC{Tv,Ti})
S.m == S.n || error("argument matrix must be square")

lu factorization of rectangular matrices is useful as a preconditioner for the normal
equation A'Ax = A'b (used for example by Wilkinson and Duff and Bjorck )

The option of using rectangular matrices is available in UMFPACK C code
and in the matlab and octave wrappers on it.

@ViralBShah ViralBShah added the domain:arrays:sparse Sparse arrays label Apr 23, 2015
@jiahao jiahao added status:help wanted Indicates that a maintainer wants help on an issue or pull request domain:linear algebra Linear algebra labels Apr 25, 2015
andreasnoack added a commit that referenced this issue Apr 27, 2015
Fix #10970. Allow LU factorizing non-square sparse matrices.
andreasnoack added a commit that referenced this issue Apr 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:arrays:sparse Sparse arrays domain:linear algebra Linear algebra status:help wanted Indicates that a maintainer wants help on an issue or pull request
Projects
None yet
Development

No branches or pull requests

3 participants