Skip to content

LinearAlgebra: 2-arg show for adjoint/transpose #54249

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

Closed
wants to merge 3 commits into from

Conversation

jishnub
Copy link
Member

@jishnub jishnub commented Apr 25, 2024

After this,

julia> t = [1,2]
2-element Vector{Int64}:
 1
 2

julia> show(transpose(t))
transpose([1, 2])

and similarly for Adjoint. This way, the displayed form is a valid constructor.

Edit: I've restricted this to cases where the parent is an AbstractVector, as this is the case where the difference between a dual vector and a matrix matters the most. A lazily transposed matrix is not all that different from the materialized form.

@jishnub jishnub added linear algebra Linear algebra arrays [a, r, r, a, y, s] labels Apr 25, 2024
@mcabbott
Copy link
Contributor

This is almost a dup of #40722

@jishnub
Copy link
Member Author

jishnub commented Apr 26, 2024

Indeed, I am happy to close this, but why was the other one stalled?

@mcabbott
Copy link
Contributor

I don't know, most things stall! Would still be nice to have, I think.

@jishnub jishnub closed this Aug 6, 2024
@giordano giordano deleted the jishnub/adjtransshow branch October 7, 2024 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arrays [a, r, r, a, y, s] linear algebra Linear algebra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants