#17004: Adding height() function to Poset
-------------------------------------+-------------------------------------
       Reporter:  jmantysalo         |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  minor              |    Milestone:  sage-6.4
      Component:  combinatorics      |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Jori Mäntysalo     |    Reviewers:  Travis Scrimshaw
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jmantysalo/adding_height___function_to_poset|  
55738f43a661ea4da60a737d31c94509a1da5de6
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Jori M\xe4ntysalo', 'oldvalue': ''}):

 * reviewer:   => Travis Scrimshaw
 * author:   => Jori Mäntysalo


Comment:

 Two typos, you're missing a "the" in front of `height` and `lenght`.
 Although I'd format the one-liner as you did in the header:
 {{{
 Return the height (the length of longest chain) of the poset.
 }}}
 but I don't really care that much. Once the typos are fixed or the
 formatting change is made, you can set this to positive review.

--
Ticket URL: <http://trac.sagemath.org/ticket/17004#comment:12>
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/d/optout.

Reply via email to