#9650: Adding support for differential forms
---------------------------------------------------------------------------+
   Reporter:  jvkersch                                                     |    
   Owner:  burcin                     
       Type:  enhancement                                                  |    
  Status:  needs_work                 
   Priority:  major                                                        |   
Milestone:  sage-4.5.3                 
  Component:  symbolics                                                    |    
Keywords:  forms, functions, symbolics
     Author:  Joris Vankerschaver                                          |    
Upstream:  N/A                        
   Reviewer:  Niles Johnson                                                |    
  Merged:                             
Work_issues:  documentation formatting, duplicates in coordinate_patch.py  |  
---------------------------------------------------------------------------+

Comment(by jvkersch):

 Hi Jason and Niles,

 Thanks for your comments, in particular for catching that code
 duplication.  I've tried to address your suggestions as well as I could,
 and here are the changes in the most recent version of the patch:

 1.  I've fixed the documentation so that it generates the output without
 any warnings, and I've also addressed the formatting issues.  This proved
 to be really tricky...

 2.  I agree on the names of the documentation files.  In the current
 patch, the documentation now reads: "Algebra of differential forms" for
 `differential_forms.py` and "Elements of the algebra of differential
 forms" for `differential_form_element.py`.  I believe that this is both
 clearer and more in line with the rest of the Sage docs, but please feel
 free to correct me on this!

 Thanks for all your time on the review!

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9650#comment:23>
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