#10637: Implement sage -sws2rst
------------------------------------------------------------+---------------
       Reporter:  nthiery                                   |         Owner:  
jason, mpatel, was                                                            
           Type:  enhancement                               |        Status:  
needs_work                                                                    
       Priority:  major                                     |     Milestone:  
sage-5.4                                                                      
      Component:  notebook                                  |    Resolution:    
                                                                            
       Keywords:  ReST, worksheet                           |   Work issues:  
answer questions, math formatting, lists, maybe absolute paths?               
Report Upstream:  Workaround found; Bug reported upstream.  |     Reviewers:  
Nicolas ThiƩry, Jason Grout, Karl-Dieter Crisman, Jason Bandlow, John Palmieri
        Authors:  Pablo Angulo, Karl-Dieter Crisman         |     Merged in:    
                                                                            
   Dependencies:  #11080, #11459                            |      Stopgaps:    
                                                                            
------------------------------------------------------------+---------------

Comment (by kcrisman):

 > No, it's the
 > 147                       'p': 'inline_no_tag',
 >       159                 'p': 'p',
 >
 > plus the:
 >
 >       331         def visit_p(self, node):
 >       332             return ''.join(self.visit(tag) for tag in
 node.contents) + '\n\n'

 Check, thanks.

 > by the way, the following lines are garbage:
 >
 >       333             cs = [self.visit(tag) for tag in node.contents]
 >       334             return ' '.join(c for c in cs if c.strip()) +
 '\n\n'
 >
 > Karl, should I make a single patch, and fix also this little stupid
 thing?

 Don't make a single patch, but do change the `response_to_kcrisman` patch
 and add this and that thing where you catch a worksheet with an empty
 title and add a title.

 > I've done nothing, I guess I could drop the code that discards the empty
 code cells, but on the other hand, I want them out of my pdf...
 Eventually one would want this to be a switch, but I guess it's ok for
 now.  That could be an enhancement request.

 Thanks for the answers to the other two questions.

 So now I should probably actually test that this works on the things I
 used it on before :) and then hopefully a pull request will be possible to
 make.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10637#comment:89>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
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