fix documentation related to ODE solvers

[robert-marik]

The documentation to ODE solvers is not written in harmony with Sage developers guide and the Sage Constructions are outdated.

Component: documentation

Author: Robert Mařík

Reviewer: David Joyner

Merged: sage-4.3.2.rc0

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

[robert-marik]

Attachment: trac_8132.patch.gz

[robert-marik]

comment:2

this patch

• fixes indentation in sage/gsl/ode.pyx from 3 spaces to 4 spaces

• fixes documentation

[robert-marik]

Attachment: trac_8132_fixed_doctests.patch.gz

fixes one failed doctest, apply on the top of previous poatch

[wdjoyner]

comment:3

This seems to be an excellent and extremely welcomed collection of docstring fixes related to solving differential equations in Sage.

Am I missing something or is it odd that calculus/desolver is not listed http://www.sagemath.org/doc/reference/modindex.html? Using
http://www.sagemath.org/doc/developer/sage_manuals.html#building-the-manuals
I see how to rebuild the manual but how do I see what the changes in the patch look like if
desolver isn't even in the manual in the first place?

Can a patch be added to include desolver in the manual?

[robert-marik]

comment:4

Fixed thanks. Thanks for the links how to do it.
Two new chapters are at the end of Symbolic Calculus.

PDF manual does not build, partly due to #8036, patly due to another problem not related to this ticket (unknown command \cross used in some file related to polynomials). The ODE part of PDF manual looks good, anyway.

Html version looks good for me.

[robert-marik]

apply on the top of the previous two patches

[robert-marik]

comment:5

Attachment: trac_8132_fixed_reference_manual.patch.gz

btw: the second problem which caused pdf not to build has been fixed by #8021

[wdjoyner]

comment:6

Three patches apply fine to 4.3.2.a0 and passes sage -testall, except for apparently unrelated failures already reported, on a mac 10.6.2.

Very nice docstring patch Robert!

Positive review.

[sagetrac-mvngu]

comment:7

Merged in the following order:

1. trac_8132.patch
2. trac_8132_fixed_doctests.patch
3. trac_8132_fixed_reference_manual.patch

[sagetrac-mvngu]

Reviewer: David Joyner

[sagetrac-mvngu]

Merged: sage-4.3.2.rc0

[sagetrac-mvngu]

Author: Robert Mařík