#16269: Cartesian Products of additive groups
-------------------------------------+-------------------------------------
       Reporter:  ncohen             |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.2
      Component:  categories         |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Nathann Cohen,     |    Reviewers:
  Nicolas M. ThiƩry                  |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  u/ncohen/16269     |  d22e245049e26dd10522d3514603868701a286fb
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by nthiery):

 Replying to [comment:18 ncohen]:
 > Here I do not agree. This is the most natural definition of an "iter"
 for a cartesian product. Your application will overrule this anyway, so
 how can that be a problem ?

 Ah shoot, in the mean time I answered your private e-mail about this
 :-) Shall I copy paste my answer here?

 > I personally see no problem in removing this catch, knowing that
 cartesian products of `FiniteEnumeratedSet` will fail. For me it is
 because of a bug in `FiniteEnumeratedSet`, not of what I implement.

 I agree.

 If this special situation is not used too much elsewhere (i.e. if all
 test pass), then I'd say let's go. And if not, I guess we will need to
 fix/workaround the problem in FiniteEnumeratedSet.

 Cheers,
                           Nicolas

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