#5308: [with patch, needs review] Removing __len__ for combinatorial classes
(depend on #5200,#4549)
---------------------------+------------------------------------------------
Reporter: hivert | Owner: hivert
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-combinat
Component: combinatorics | Keywords: __len__, count, combinatorial class
---------------------------+------------------------------------------------
Comment(by saliola):
Replying to [comment:9 hivert]:
> Dear Franco,
>
> I can't reproduce the problem on word but I don't have access to a 32bit
machine. More precisely, {{{words.size_of_alphabet()}}} seems now to
always return a sage integer. That should fix the problem. Can you please
check it ?
Sorry it took so long: I had to compile sage on a 32-bit machine!
The last patch
(combinatorialclass_iter_len_count_cleanup-5308-submitted.patch) applies
cleanly to sage-3.4 on top of the patches for #5200 and #4549. I ran all
the combinat doctests and all tests passed.
You get a positive review from me. I won't change the status though, since
Nicolas might still have some issues.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5308#comment:10>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
