#14826: Newton polygons
-----------------------------------+-----------------------------
       Reporter:  caruso           |         Owner:  roed
           Type:  enhancement      |        Status:  needs_review
       Priority:  major            |     Milestone:  sage-5.12
      Component:  padics           |    Resolution:
       Keywords:  Newton polygons  |     Merged in:
        Authors:  Xavier Caruso    |     Reviewers:
Report Upstream:  N/A              |   Work issues:
         Branch:                   |  Dependencies:
       Stopgaps:                   |
-----------------------------------+-----------------------------

Comment (by vbraun):

 Try the `@cached_method` decorator instead of caching manually.

 The docstrings should start with a short (ideally one-line) description
 {{{
 def vertices(self, copy=True):
     """
     Return the vertices of the Newton polytope.

     INPUT:
     ...
 }}}

 Are you going to add a method to polynomaials or is that going to be in a
 future ticket?

 You should run the testsuite for parents and elements somewhere in your
 code (see
 
http://www.sagemath.org/doc/reference/misc/sage/misc/sage_unittest.html#sage.misc.sage_unittest.TestSuite)

--
Ticket URL: <http://trac.sagemath.org/ticket/14826#comment:11>
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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to