On 2/17/12 12:13 AM, Florent Hivert wrote:
>>> By the way: changing P.linear_extensions() to return an enumerated set
>>> rather than a plain list could make sense, assuming there is no speed
>>> loss. This might require a bit of cythonification; maybe using
>>> ClonableArray will be enough. It's slightly backward incompatible (it
>>> does not print the same).
>>
>> Why did you make the choice to print differently in ClonableArray as opposed 
>> to CombinatorialObject?
> 
> What is the difference ?
> 
> sage: CombinatorialObject([1,2,3])
> [1, 2, 3]
> 
> sage: class toto(ClonableIntArray):
> ....:     def check(self): pass
> sage: toto(Parent(), [1,2,3])
> [1, 2, 3]

Which patch is ClonableIntArray in? For me it is not defined (and I am hesitant 
to base my
code on something not yet in sage):

sage: class toto(ClonableIntArray):
....:     def check(self): pass
....:
---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)

/Applications/sage-5.0.beta3/devel/sage-combinat/sage/combinat/posets/<ipython 
console> in <module>()

NameError: name 'ClonableIntArray' is not defined

Best,

Anne

-- 
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.

Reply via email to