#10541: Conversions of vectors to matrices
------------------------------------------+---------------------------------
Reporter: rbeezer | Owner: jason, was
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-4.6.2
Component: linear algebra | Keywords:
Author: Rob Beezer | Upstream: N/A
Reviewer: Volker Braun, Marco Streng | Merged:
Work_issues: |
------------------------------------------+---------------------------------
Comment(by mstreng):
Replying to [comment:11 rbeezer]:
> So I feel strongly that just having a "vector transpose" is a source of
confusion,
Hi Rob,
Thanks for the explanation. I know most of that already, and personally do
agree with it.
On the other hand, some people may have used vector {{{transpose}}} a lot
in their own code, or be strongly opposed to this change for other
reasons. You did ask if there were any objections, but that was in the
middle of a long thread with title 'Adjoint of a matrix'. I'm not sure if
people got enough chance to object.
> I do not think the doctest in question is incorrect. It wasn't failing
was it?
It wasn't incorrect in that sense, it just occurred twice, and tested the
wrong function in the second occurrence. Both {{{.row()}}} and
{{{.column()}}} have the same doctest
{{{
sage: v = vector(ZZ, [])
sage: w = v.row()
sage: w.parent()
Full MatrixSpace of 1 by 0 dense matrices over Integer Ring
}}}
It would make more sense to replace {{{v.row()}}} by {{{v.column()}}} in
the doctest of {{{column}}} instead of repeating the doctest of {{{row}}}.
I assumed that is what you meant to do in your first patch, so it is what
I did in mine.
Marco
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10541#comment:14>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
