"Nicolas M. Thiery" <nicolas.thi...@u-psud.fr> writes: > Hi Martin! > > On Tue, Mar 13, 2012 at 09:53:08AM +0100, Martin Rubey wrote: >> I just noticed that >> >> Partitions(-1).list() # or any negative integer >> >> gives a maximum recursion depth exceeded error. I think it should >> return the empty list, right? > > Yeah, or probably simply throw a ValueError?
In the code at hand I'd prefer to have the empty list. Currently I need to have a case distinction... With keyword argument min_part, we also get the empty list, by the way. > If yes, you are welcome to create ticket, this is now #12663 Martin > and post a patch adding a non-negativity test line 3341 of > sage.combinat.partition.py, around: > > if isinstance(n, (int,Integer)): > return Partitions_n(n) > else: > raise ValueError, "n must be an integer" > > together with an appropriate doctest. > > Cheers, > Nicolas > -- > Nicolas M. ThiƩry "Isil" <nthi...@users.sf.net> > http://Nicolas.Thiery.name/ -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To post to this group, send email to sage-combinat-devel@googlegroups.com. To unsubscribe from this group, send email to sage-combinat-devel+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/sage-combinat-devel?hl=en.