#18640: Topological manifolds: scalar fields
-------------------------------------+-------------------------------------
       Reporter:  egourgoulhon       |        Owner:  egourgoulhon
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-7.2
      Component:  geometry           |   Resolution:
       Keywords:  topological        |    Merged in:
  manifolds                          |    Reviewers:
        Authors:  Eric Gourgoulhon,  |  Work issues:
  Michal Bejger                      |       Commit:
Report Upstream:  N/A                |  57b7b7b90110d1e4cc6d7e612f43cab49efc80c8
         Branch:                     |     Stopgaps:
  public/manifolds/top_manif_scalar_fields|
   Dependencies:  #18529             |
-------------------------------------+-------------------------------------
Changes (by tscrim):

 * status:  needs_info => needs_review


Comment:

 Wee, finally had time to get it all done. I started to use {{{``self``}}}
 because I recall less consensus on the sage-devel thread; a good chuck of
 code uses it; if we add more code, it becomes standard; and it simplifies
 a number of the doc sentences' structure. I did not implement as much
 coercion as might be possible (i.e., coordinate functions could coerce to
 the smallest chart they are defined on), but the door is open to do so. I
 also added the global options and marked the tests in `utilities.py` as `#
 known bug`. So if my changes are good with you, then we can set this to a
 positive review.

--
Ticket URL: <http://trac.sagemath.org/ticket/18640#comment:37>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to