#10543: Echelon form over QQ is mutable
------------------------------+---------------------------------------------
   Reporter:  rbeezer         |       Owner:  jason, was     
       Type:  defect          |      Status:  positive_review
   Priority:  minor           |   Milestone:  sage-4.6.2     
  Component:  linear algebra  |    Keywords:                 
     Author:  Rob Beezer      |    Upstream:  N/A            
   Reviewer:  Tom Coates      |      Merged:                 
Work_issues:                  |  
------------------------------+---------------------------------------------
Changes (by rbeezer):

  * status:  needs_work => positive_review


Old description:

> Which is not the norm for matrices over other rings.
>
> {{{
> sage: A=matrix(QQ,2,range(4))
> sage: B=matrix(ZZ,2,range(4))
> sage: C=A.echelon_form()
> sage: D=B.echelon_form()
> sage: C.is_mutable()
> True
> sage: D.is_mutable()
> False
> }}}
>
> See http://groups.google.com/group/sage-
> devel/browse_thread/thread/ef90d24da0ee704c
>
> Apply
>
> trac_10543-rational-echelon-form-immutable.patch, trac_10543-reviewer-
> rebased.patch

New description:

 Which is not the norm for matrices over other rings.

 {{{
 sage: A=matrix(QQ,2,range(4))
 sage: B=matrix(ZZ,2,range(4))
 sage: C=A.echelon_form()
 sage: D=B.echelon_form()
 sage: C.is_mutable()
 True
 sage: D.is_mutable()
 False
 }}}

 See http://groups.google.com/group/sage-
 devel/browse_thread/thread/ef90d24da0ee704c

 Apply

 trac_10543-rational-echelon-form-immutable-v2.patch, trac_10543-reviewer-
 rebased.patch

--

Comment:

 Applied #10028 to 4.6.2.alpha3, only got one failure when applying the
 first of the two patches here.  Rebased that patch only, though the
 failure seems trivial.  All tests pass in sage/matrix2.

 If this does not work, we'll go for alpha4.

 Apply "v2" of original patch, then "reviewer-rebased," in that order.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10543#comment:8>
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.

Reply via email to