#15132: Please remove matrix.copy()
-----------------------------------------+----------------------------
Reporter: ppurka | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-5.12
Component: linear algebra | Resolution:
Keywords: | Merged in:
Authors: Punarbasu Purkayastha | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
-----------------------------------------+----------------------------
Comment (by ppurka):
Replying to [comment:3 kcrisman]:
> Just a question for the author - are the `copy(m)`-style tests (or
''very'' similar ones) anywhere else in Sage (e.g. in the {{{__copy__}}}
method)? It would be unfortunate to lose those.
It seemed to me yesterday that all the examples were present in some form
in the other files. You can run something like
{{{
$ cd SAGE_ROOT/devel/sage/sage/matrix
$ grep -C10 'def __copy__' *.py*
}}}
to see some of the doctests. The last test in the `copy()` method is
actually a bit strange and so I have copied that code into the
`matrix_generic_dense.pyx` file. The
[attachment:trac_15132-remove_matrix_copy.patch patch] has been
accordingly updated.
--
Ticket URL: <http://trac.sagemath.org/ticket/15132#comment:5>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
