#17108: IncidenceStructure.degree(set={1,2,3})
-------------------------------------+-------------------------------------
       Reporter:  ncohen             |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.4
      Component:  combinatorial      |   Resolution:
  designs                            |    Merged in:
       Keywords:                     |    Reviewers:
        Authors:  Nathann Cohen      |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  u/ncohen/17108     |  c34fda620e898708392becbbf8994d9065e29371
   Dependencies:  #17104             |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by vdelecroix):

 Replying to [comment:6 ncohen]:
 > > I meant instead of `set`; use something else like `points` or `pts`.
 >
 > That's what I understood. Right now we have 'point', it would be
 dangerous to have 'points' and weird to have 'pts'.

 Right now
 {{{
 - def degree(self, p=None):
 + def degree(self, p=None, set=None, sets_of_size=None):
 }}}
 There is no `point`...

 > > Note that I assumed that the following is ugly
 >
 > What does this code do ?

 So you agree that it is unreadable ;-)

 Vincent

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