-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
add 'ᵀ postfix operator for transpose #38062
Conversation
Why was |
It's already a valid identifier and identifiers like |
Triage approved of this, so should be ready to go. |
Just read the 1.6 news, to me, it definitely looks like a conjugate rather than a transpose. |
Just saw this in the 1.6 changelog too, and to me (math/physics at ETH Zurich), Imho 1.6 should be released without this, and a discussion reopened later. |
I didn't see this change until now either, I second what @GiggleLiu @dnadlinger says a'ᵀ does look more like transpose+adjoint. |
I've opened #40070 for further discussion. |
This reverts commit 665279a.
…iaLang#40075) This reverts commit 665279a.
…iaLang#40075) This reverts commit 665279a.
I feel like
transpose
is really the only sensible meaning of'ᵀ
and this was also the main use case for #37247, so I think defining it in Base makes the most sense.fixes #21037