#18849: category of lazy family wrong if infinite combinatorial class used as 
index
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  cnassau                |       Status:  needs_work
           Type:         |    Milestone:  sage-6.8
  defect                 |   Resolution:
       Priority:  major  |    Merged in:
      Component:         |    Reviewers:  Vincent Delecroix
  categories             |  Work issues:
       Keywords:         |       Commit:
        Authors:         |  09f97fa45de98e4ede496f7f5401cdd84bdda05f
  Christian Nassau       |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/cnassau/18849        |
   Dependencies:         |
-------------------------+-------------------------------------------------
Changes (by vdelecroix):

 * status:  needs_review => needs_work
 * reviewer:   => Vincent Delecroix


Comment:

 Hello,

 1. Why a special care for `CombinatorialClass` is even needed at all? The
 following would be much simpler
 {{{
 category = EnumeratedSets()
 if set in Sets().Finite() or isinstance(set, (list,tuple)):
     category = category.Finite()
 elif set in Sets().Infinite():
     category = category.Infinite()
 }}}

 2. Your documentation lines are too long.

 Vincent

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