#10424: Expand matrix augment'ing to allow vectors as input
------------------------------+---------------------------------------------
Reporter: rbeezer | Owner: jason, was
Type: enhancement | Status: needs_work
Priority: minor | Milestone: sage-4.6.1
Component: linear algebra | Keywords:
Author: Rob Beezer | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Changes (by rbeezer):
* status: needs_review => needs_work
Comment:
Replying to [comment:3 robertwb]:
> The current augment doesn't really have any optimized functionality, so
I think it'd make more sense to put this all into a single function. Just
do
>
> cdef Matrix other
>
> and name the original input to be something else.
Yes, I was wrestling with the `cdef` for `other` and didn't want to slow
down any of the optimized stuff there. It would be better to make this
all one function. So I'll take your hint and try to make it happen.
I did notice that matrix row operations are implemented in two parts (a
pure python routine and a Cython routine), but maybe that is because the
Cython part gets used twice typically.
Thanks for the assist and I'll get back to this soon.
Rob
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10424#comment:4>
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.
