Opened 11 years ago
Closed 9 years ago
#8046 closed defect (fixed)
Add matrix/matrix_double_dense.py to documentation
Reported by: | rbeezer | Owned by: | mvngu |
---|---|---|---|
Priority: | major | Milestone: | sage-5.0 |
Component: | documentation | Keywords: | beginner sd35.5 |
Cc: | Merged in: | sage-5.0.beta1 | |
Authors: | Kenneth Smith, John Palmieri | Reviewers: | John Palmieri, Karl-Dieter Crisman |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
The source file matrix/matrix_double_dense.py
is not included in the documentation. It appears that it should be, since it has functions that are of interest to users. Patch simply adds it to the right place in the documentation tree.
The file itself needs some love. After #4756 goes in, the following four functions should be in good shape. The remainder needs work.
left_eigenvectors() right_eigenvectors() eigenspaces_left() eigenspaces_right()
Apply trac_8046_matrix_double_dense.patch and trac_8046-ref.patch.
Attachments (3)
Change History (14)
Changed 11 years ago by
comment:1 follow-up: ↓ 2 Changed 10 years ago by
- Status changed from new to needs_work
comment:2 in reply to: ↑ 1 Changed 10 years ago by
Replying to jason:
Yes, the file "needs_work". I went through it a couple days ago and cleaned up lots of little things (documentation mostly), but then went off and made a few patches with code changes. I'll get back to it very soon and insert into the other work I'm doing.
comment:3 Changed 9 years ago by
- Keywords beginner sd35.5 added
comment:4 Changed 9 years ago by
I made a new patch. This one includes the matrix double dense to the documentation like the last one, and it also edits the file a lot to get rid of almost all of the syntax errors. I am still getting 2 warnings however:
One is:
docstring of sage.matrix.matrix_double_dense:8: (WARNING/2) Bullet list ends without a blank line; unexpected unindent.
I cannot, however find what the error message is talking about, even after multiple people examined it for quite a while.
The other one is:
WARNING: dvipng command 'dvipng' cannot be run (needed for math display), check the pngmath_dvipng setting
which I am told can be ignored.
So really there is just one warning that could still be fixed.
Changed 9 years ago by
comment:5 Changed 9 years ago by
- Description modified (diff)
I'm attaching a 'referee' patch to fix up some docstrings. The first change in that patch fixes the warning message about the unexpected unindent; the others just tidy some things up.
Changed 9 years ago by
comment:6 Changed 9 years ago by
- Status changed from needs_work to needs_review
comment:7 Changed 9 years ago by
- Reviewers set to John Palmieri, Karl-Dieter Crisman, Jason Grout, Jeroen Demeyer, Steven Trogdon, Leif Leonhardy, Sébastien Labbé
- Status changed from needs_review to positive_review
Looks great! Thanks for catching this, John.
comment:8 follow-up: ↓ 9 Changed 9 years ago by
If you build the documentation with "-j", it will use jsmath, and should not give the error about dvipng: sage -docbuild reference html -j
comment:9 in reply to: ↑ 8 Changed 9 years ago by
- Reviewers changed from John Palmieri, Karl-Dieter Crisman, Jason Grout, Jeroen Demeyer, Steven Trogdon, Leif Leonhardy, Sébastien Labbé to John Palmieri, Karl-Dieter Crisman
If you build the documentation with "-j", it will use jsmath, and should not give the error about dvipng:
sage -docbuild reference html -j
Interesting. In any case, that was clearly an unrelated error.
comment:10 Changed 9 years ago by
- Milestone changed from sage-4.8 to sage-5.0
comment:11 Changed 9 years ago by
- Merged in set to sage-5.0.beta1
- Resolution set to fixed
- Status changed from positive_review to closed
I get some errors on 4.6.1 when building the docs: