#10349: Implementation of mutations for matrices
-----------------------------+----------------------------------------------
Reporter: stumpc5 | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.0
Component: combinatorics | Keywords: matrix mutation
Work_issues: | Upstream: N/A
Reviewer: Hugh Thomas | Author: Christian Stump
Merged: | Dependencies: #10347
-----------------------------+----------------------------------------------
Old description:
> This patch implements a method to mutate matrices. This method is needed
> for the cluster algebra and quiver package, see Ticket #10298.
New description:
This patch implehttp://www.python.org/dev/peps/pep-0353/ments a method to
mutate matrices. This method is needed for the cluster algebra and quiver
package, see Ticket #10298.
--
Comment(by stumpc5):
Replying to [comment:13 hthomas]:
> Hi Christian--
>
> What's the point of declaring k to be of type Py_ssize_t? I thought the
point was that this would allow k to be huge, but in that case, it seems
that you would get overflows when you run into the fact that i and j are
declared to be integers.
Hi Hugh,
`Py_ssize_t` is the recommended integer type for cython, see
http://www.python.org/dev/peps/pep-0353/. In fact, in our current
situation there is no advantage of using this type over using `int`. It is
used all over in the code for matrices, see for example the method
`get_unsafe` - that's why I also used it.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10349#comment:15>
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.