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.

Sign up for GitHub

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

Jump to bottom

Pinv of lazy adjoints and transposes? #29723

Closed
saolof opened this issue Oct 19, 2018 · 0 comments
Closed

Pinv of lazy adjoints and transposes? #29723

saolof opened this issue Oct 19, 2018 · 0 comments
Labels
linear algebra Linear algebra

Comments

@saolof
Copy link

saolof commented Oct 19, 2018 •
edited
Loading

The two methods for pinv of lazy transposes/adjoints in stdlib/LinearAlgebra/src/adjtrans.jl only apply to the transpose or adjoint of a vector, but not matrices.

The Moore-Penrose pseudoinverse commutes with both conjugation, transposes, and conjugate transposes (adjoints). A naive implementation of pinv for Adjoints would be pinv(m::Adjoint) = pinv(m')' , and similarly with transpose for lazy transposes.
@saolof saolof changed the title Pinv of adjoint and transpose? Pinv of lazy adjoints and transposes? Oct 19, 2018
@kshyatt kshyatt added the linear algebra Linear algebra label Oct 19, 2018
andreasnoack added a commit that referenced this issue Oct 22, 2018
@andreasnoack
Make pinv work for Adjoint
feaff4f 
Fixes #29723
andreasnoack added a commit that referenced this issue Oct 29, 2018
@andreasnoack
Make pinv work for Adjoint
a60d158 
Fixes #29723
@andreasnoack andreasnoack mentioned this issue Oct 29, 2018
Merged
andreasnoack added a commit that referenced this issue Oct 29, 2018
@andreasnoack
Make pinv work for Adjoint
44417bd 
Fixes #29723
andreasnoack added a commit that referenced this issue Oct 30, 2018
@andreasnoack
Make pinv work for Adjoint (#29837)
a472bc7 
Fixes #29723
KristofferC pushed a commit that referenced this issue Oct 31, 2018
@andreasnoack @KristofferC
Make pinv work for Adjoint (#29837)
0afb550 
Fixes #29723

(cherry picked from commit a472bc7)
KristofferC pushed a commit that referenced this issue Oct 31, 2018
@andreasnoack @KristofferC
Make pinv work for Adjoint (#29837)
f938959 
Fixes #29723

(cherry picked from commit a472bc7)
KristofferC pushed a commit that referenced this issue Nov 2, 2018
@andreasnoack @KristofferC
Make pinv work for Adjoint (#29837)
bb6a9af 
Fixes #29723

(cherry picked from commit a472bc7)
KristofferC pushed a commit that referenced this issue Feb 11, 2019
@andreasnoack @KristofferC
Make pinv work for Adjoint (#29837)
908d9cd 
Fixes #29723

(cherry picked from commit a472bc7)
KristofferC pushed a commit that referenced this issue Feb 20, 2020
@andreasnoack @KristofferC
Make pinv work for Adjoint (#29837)
64d20d2 
Fixes #29723

(cherry picked from commit a472bc7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
linear algebra Linear algebra
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kshyatt @saolof