matrix indexing should be explained in the manual

[jasongrout]

A lot of the great stuff in the docstrings for the matrix __getitem__ and __setitem__ functions should be in the reference manual somewhere, maybe http://sagemath.org/doc/reference/sage/matrix/docs.html.

CC: @rbeezer @sagetrac-mvngu

Component: documentation

Author: Jason Grout

Reviewer: Robert Bradshaw, Minh Van Nguyen

Merged: sage-4.3.4.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/7877

[jasongrout]

comment:1

Attachment: trac-7877-doc-matrix-indexing.patch.gz

[jasongrout]

Author: Jason Grout

[robertwb]

comment:3

Depends on #8008, but otherwise, excellent!

[sagetrac-mvngu]

Attachment: trac_7877-reviewer.patch.gz

apply on top of previous

[sagetrac-mvngu]

Reviewer: Robert Bradshaw, Minh Van Nguyen

[sagetrac-mvngu]

comment:5

Here's a typo in the patch:

96	Get the second column of M:: 
97	         
98	    sage: M[1:,0] 
99	    [ 1] 
100	    [ 1] 
101	    [-1]

The code should be:

sage: M[:,1]
[-2]
[ 8]
[ 1]
[ 2]

I have attached patch for this, which needs some review. Otherwise the whole ticket looks good.

[jasongrout]

Attachment: trac-7877-doc-error.patch.gz

[jasongrout]

comment:6

positive review for Minh's patch; I attached the same patch for the __getitem__ docstring in matrix0.pyx.

[jasongrout]

comment:7

(apply trac-7877-doc-error.patch on top of all previous patches)

[sagetrac-mvngu]

Merged: sage-4.3.4.alpha0

[sagetrac-mvngu]

comment:8

Merged in this order:

1. trac-7877-doc-matrix-indexing.patch
2. trac_7877-reviewer.patch
3. trac-7877-doc-error.patch

Jason: You should put a sensible commit message in your patch, together with the ticket number.