Bring doctests of modular/modsym/p1list.py up to 100%

[JohnCremona]

The attached patch completes all docstrings and doctests for the file modular/modsym/p1list.py. I also checked that the html and pdf output in the reference manual look good.

CC: @loefflerd

Component: modular forms

Keywords: modular form documentation

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

[JohnCremona]

Attachment: p1list-doc.patch.gz

Based on 3.4.1.rc2

[loefflerd]

comment:2

Patch applies fine to 3.4.1.rc2, all doctests pass, and reference manual builds happily. There is just one tiny typo: in the docstring for "apply_S", it has "[-0,1;1,0]" instead of "[0,-1;1,0]". I have uploaded a patch that corrects this.

[loefflerd]

apply over previous patch

[JohnCremona]

comment:3

Attachment: trac_5770-reviewer_fix.patch.gz

Thanks, David. Getting the documentation to build & look ok takes a long time! As you can imagine there was a lot of cutting and pasting. Much more of the same to com with manin_symbols.py, where there are about 5 classes each of which has a very similar set of methods (but not quite identical).

[sagetrac-mabshoff]

comment:4

Merged both patches in Sage 3.4.1.rc3.

Cheers,

Michael