#11528: Allow deleting row or column from matrix
----------------------------------+-----------------------------------------
Reporter: kcrisman | Owner: jason, was
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-5.1
Component: linear algebra | Resolution:
Keywords: delete remove | Work issues:
Report Upstream: N/A | Reviewers: Rob Beezer
Authors: Wai Yan Pong | Merged in:
Dependencies: | Stopgaps:
----------------------------------+-----------------------------------------
Old description:
> This was requested in the 2011 Sage MAA PREP workshop. Names deleted.
> {{{
> : Is there any way to delete specified rows or columns from an existing
> matrix?
> : Hmm, I think that is possible, using a method.
> : I'll look.
> : - we can swap rows, scale rows, add mults of rows, create a matrix
> from specified rows... but not that.
> : Would you find this useful? Is it in Matlab?
> : For list there is a method called pop
> : Right, but not for matrices.
> : if matrix is a list of list...
> : But it's not quite that.
> : I'l make a ticket.
> : this would be useful
> : there is an m.insert_row
> : Hah! But we can't do it backwards
> }}}
> So let's make this happen.
>
> '''Apply''':
> 1. [attachment:trac_11528_delete_rows_and_columns.path.patch]
> 1. [attachment:trac_11528_matrix_delete_rows_and_columns.patch]
New description:
This was requested in the 2011 Sage MAA PREP workshop. Names deleted.
{{{
: Is there any way to delete specified rows or columns from an existing
matrix?
: Hmm, I think that is possible, using a method.
: I'll look.
: - we can swap rows, scale rows, add mults of rows, create a matrix from
specified rows... but not that.
: Would you find this useful? Is it in Matlab?
: For list there is a method called pop
: Right, but not for matrices.
: if matrix is a list of list...
: But it's not quite that.
: I'l make a ticket.
: this would be useful
: there is an m.insert_row
: Hah! But we can't do it backwards
}}}
So let's make this happen.
'''Apply''':
1. [attachment:trac_11528_delete_rows_and_columns.path.patch]
1. [attachment:trac_11528_matrix_delete_rows_and_columns.patch]
1. [attachment:trac_11528_matrix_delete_rows_columns.patch]
1. [attachment:trac_11528-reviewer.patch]
--
Comment (by rbeezer):
Needs two things for sure.
1. `.::` will make a period and a single colon. One or the other, but
we don't want both. The space you removed needs to be there.
1. First error message should be tested.
I have made a reviewer patch addressing these and took the liberty of
improving an error message and making the `SEEALSO` into a complete
sentence.
I still need to run tests again on a newer version of Sage - on 4.8 at the
office right now.
Pong - you can review my "reviewer patch" and indicate if you agree or
not. '''Somebody''' will have to review it. If tests are successful I
will
(a) go to positive review
(b) fold all the patches into one for the convenience of the release
manger.
Right now this needs all 4 patches present.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11528#comment:22>
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.
