Opened 13 years ago
Closed 13 years ago
#8132 closed defect (fixed)
fix documentation related to ODE solvers
Reported by: | Robert Marik | Owned by: | Minh Van Nguyen |
---|---|---|---|
Priority: | minor | Milestone: | sage-4.3.2 |
Component: | documentation | Keywords: | |
Cc: | Merged in: | sage-4.3.2.rc0 | |
Authors: | Robert Mařík | Reviewers: | David Joyner |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
The documentation to ODE solvers is not written in harmony with Sage developers guide and the Sage Constructions are outdated.
Attachments (3)
Change History (10)
Changed 13 years ago by
Attachment: | trac_8132.patch added |
---|
comment:1 Changed 13 years ago by
Status: | new → needs_review |
---|
comment:2 Changed 13 years ago by
Changed 13 years ago by
Attachment: | trac_8132_fixed_doctests.patch added |
---|
fixes one failed doctest, apply on the top of previous poatch
comment:3 Changed 13 years ago by
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?
comment:4 Changed 13 years ago by
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.
Changed 13 years ago by
Attachment: | trac_8132_fixed_reference_manual.patch added |
---|
apply on the top of the previous two patches
comment:5 Changed 13 years ago by
btw: the second problem which caused pdf not to build has been fixed by #8021
comment:6 Changed 13 years ago by
Status: | needs_review → positive_review |
---|
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.
comment:7 Changed 13 years ago by
Authors: | → Robert Mařík |
---|---|
Merged in: | → sage-4.3.2.rc0 |
Resolution: | → fixed |
Reviewers: | → David Joyner |
Status: | positive_review → closed |
Merged in the following order:
this patch