#6250: [with patch, needs review] Standardize MatrixGroup_gap: by adding
.cardinality, and deprecating __len__
---------------------+------------------------------------------------------
Reporter: nthiery | Owner: nthiery
Type: defect | Status: assigned
Priority: major | Milestone: sage-4.0.2
Component: algebra | Keywords: cardinality, __len__, order, groups
Reviewer: | Author: nthiery
Merged: |
---------------------+------------------------------------------------------
Changes (by nthiery):
* status: new => assigned
Comment:
Replying to [comment:4 wdj]:
> I don't agree with the suggestion in one of the docstrings that order
might be deprecated.
> But that is just my (American) opinion, which might not be shared by the
rest of the world:-)
I am fine with both options. From discussions on sage-devel, it seems that
in general aliases are somewhat frowned upon.
We definitely want .cardinality(). But yes, even in Europe, some users
would certainly be trying G.order() to get the size
of the group. That's why I raised the issue.
I am happy to remove the comment if you think its better.
> In any case, the patches do not apply cleanly to 4.0.rc0.
? I just retried, and it applies smoothly on sage 4.0.1. Did you only
apply the second patch? (the first one should be deleted)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6250#comment:5>
Sage <http://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 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
-~----------~----~----~----~------~----~------~--~---
