improve doctest coverage of matrix/benchmark.py from 0% to 100%

[williamstein]

Part of metaticket #12024

Apply:

• attachment: trac_12034.patch
• attachment: trac_12034_reviewer.patch
• attachment: trac_12034-reviewer-reviewer.patch

CC: @malb

Component: linear algebra

Author: William Stein

Reviewer: Martin Albrecht, Karl-Dieter Crisman

Merged: sage-5.0.beta4

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

[williamstein]

Attachment: trac_12034.patch.gz

[williamstein]

Author: wstein

[malb]

comment:3

Attachment: trac_12034_reviewer.patch.gz

I've added a reviewer patch which adds INPUT: blocks and fixes some wrongly stated default parameters.

[kcrisman]

comment:4

Martin, does that mean you give positive review to William's patch? It looks fine at a glance... Anyway, putting "needs info" for this.

Other than not obeying the recommendation from the developer guide about .. note

However, you are encouraged to use the upper-case version

which is not a problem at all, the reviewer patch looks fine, including the very minor code changes. Just one or two very minor formatting things.

I've attached a reviewer-reviewer patch which fixes these and adds this to the reference manual.

[kcrisman]

Reviewer: Martin Albrecht, Karl-Dieter Crisman

[kcrisman]

Changed author from wstein to William Stein

[kcrisman]

comment:5

Attachment: trac_12034-reviewer-reviewer.patch.gz

[malb]

comment:6

I'm fine with William's and your patches.

[jdemeyer]

Merged: sage-5.0.beta4