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

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jishnub
Copy link
Contributor

@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 domain:linear algebra Linear algebra domain: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
Contributor Author

jishnub commented Apr 26, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:arrays [a, r, r, a, y, s] domain:linear algebra Linear algebra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants