Make (c)transpose work correctly for block matrices#7244
Conversation
|
This seems to me like the right thing to do. |
|
Oh. I didn't run the string tests locally. I'll define |
|
lgtm |
|
Or, I guess |
|
I know it seems a bit sketch, but I think this is the way to go. Let's try it out and see what comes of it. |
|
I'll do that for now, but defining methods for |
|
I have sometimes wondered if we should add This is a bit sketch, but less so than making it work only for strings. |
|
Okay. I have added line definitions to |
|
I like Atom and Collection, but which does a String qualify as? |
|
Yep, that's one reason I didn't add those types :) You don't necessarily want to be forced to classify everything that way. |
|
I like this one. |
|
lgtm too |
Make (c)transpose work correctly for block matrices
With this change, the matrix
Apdgenerated bywill be positive definite.