Ticket #10543: trac_10543-reviewer-rebased.patch

File trac_10543-reviewer-rebased.patch, 2.1 KB (added by rbeezer, 9 years ago)
  • sage/matrix/constructor.py

    # HG changeset patch
    # User Tom Coates <tomc@imperial.ac.uk>
    # Date 1296684119 0
    # Node ID 79dc4704853ea9af8ad74bf3a43ef7fdf66a8212
    # Parent  1a890084f4125b943bb8e9745a76356a7696bfdc
    trac 10543: reviewer patch
    
    diff -r 1a890084f412 -r 79dc4704853e sage/matrix/constructor.py
    a b  
    25842584        [  0   0   0   0   0   0   0   0   1   0  -5   0  -3   2]
    25852585        [  0   0   0   0   0   0   0   0   0   1   1   0   1  -3]
    25862586        [  0   0   0   0   0   0   0   0   0   0   0   1  -1   1]
    2587         sage: B_expanded.set_immutable()
     2587
     2588    Check that we fixed Trac #10543 (echelon forms should be immutable)::
     2589
     2590        sage: B_expanded.is_immutable()
     2591        True
     2592
     2593    We want to modify B_expanded, so replace it with a copy::
     2594
     2595        sage: B_expanded = copy(B_expanded)
    25882596        sage: B_expanded.subdivide(B.nrows()-B.nullity(),B.ncols());B_expanded # random
    25892597        [  1   3   0   0   1   1   3  -2|  0   0  -3   0  -9  16]
    25902598        [  0   0   1   0   3  -2  -1  -3|  0   0   2   0  11 -27]
  • sage/matrix/matrix_rational_dense.pyx

    diff -r 1a890084f412 -r 79dc4704853e sage/matrix/matrix_rational_dense.pyx
    a b  
    14451445           proof=True.
    14461446       
    14471447       
    1448         OUTPUT: self is no in reduced row echelon form.
     1448        OUTPUT: the reduced row echelon form of self.
    14491449       
    14501450        EXAMPLES::
    14511451       
     
    14651465            [      0       0       1 238/157]
    14661466            [      0       0       0       0]
    14671467
    1468         The result is an immutable matrix, so a copy is necessary
    1469         if you want a result you can change somehow.  This exercises a
    1470         fix for Trac #10543.
     1468        The result is an immutable matrix, so if you want to
     1469        modify the result then you need to make a copy.  This
     1470        checks that Trac #10543 is fixed. ::
    14711471
    14721472            sage: A = matrix(QQ, 2, range(6))
    14731473            sage: E = A.echelon_form()