#5330: [with patch, needs work] Move the docs over to the main repository
---------------------------+------------------------------------------------
 Reporter:  mhansen        |       Owner:  mhansen 
     Type:  enhancement    |      Status:  assigned
 Priority:  major          |   Milestone:  sage-3.4
Component:  documentation  |    Keywords:          
---------------------------+------------------------------------------------

Comment(by jhpalmieri):

 A few comments:

 in matrices.rst, {{{``sage/matrix/matrixrealdoubledense.py``}}} should be
 {{{``sage/matrix/matrix_real_double_dense.pyx``}}}.  (Note the trailing
 'x', as well as the underscores.)

 in databases.rst: this is a pre-existing condition, but in these lines:
 {{{
 Supports databases up to 2 tebibytes (241 bytes) in size.

 Strings and BLOBs up to 2 gibibytes (231 bytes) in size.
 }}}
 I think the numbers should be {{{2^41}}} and {{{2^31}}}, respectively.

 in interfaces.rst: I think that paragaphs 2 and 3 (the two indented
 paragraphs) should be denoted by {{{.. note::}}}, or whatever the ReST
 syntax is.

 in interfaces.rst: another pre-existing condition: change "esp., useful"
 to "especially useful"

 in interfaces.rst: I think that at least the html version would look
 better if {{{Chapter :ref:`ch:libraries`}}} were changed to just
 {{{:ref:`ch:libraries`}}}.  This occurs twice in the file.

 Oh dear, I can't find intro.rst here.  Well, wherever the file which used
 to be 'ref/intro.tex' is, it has a similar issue: in the html, "Chapter
 The Sage Command Line" would look better as "The Sage Command Line", and
 the same for "Chapter The Sage Notebook".


 (This is only a review of the files which used to be 'ref/blah.tex' -- I
 didn't look at any other parts of the various patches here.)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5330#comment:6>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to