#14319: Automorphism group with labeled vertices
-------------------------------------------------+-------------------------
Reporter: ncohen | Owner: jason,
Type: enhancement | ncohen, rlm
Priority: major | Status: closed
Component: graph theory | Milestone: sage-5.10
Keywords: | Resolution: fixed
Authors: Nathann Cohen | Merged in:
Report Upstream: N/A | sage-5.10.beta2
Branch: | Reviewers: Volker
Dependencies: #14291, #14250, #14477, | Braun
#14435 | Work issues:
| Commit:
| Stopgaps:
-------------------------------------------------+-------------------------
Comment (by nthiery):
Replying to [comment:10 ncohen]:
> Patch updated ! I did not touch `list()`. What do you think we should do
with it ? Update its docstring to say "returns the domain" ? Looks like
this is all it is used for. And hopefully, because doing otherwise would
assume that it is only defined on integers.
Damned, I missed this. The list method is important! It gives the
permutation in list notation, and I am using it in my class tomorrow! Ok,
the meaning is a bit dubious when the domain is any set, but I guess the
following semantic would make sense:
- domain(): returns self.parent().domain() (if at all needed)
- list(): returns [self(i) for i in self.parent().domain()]
- tuple(): returns tuple(self.list())
If you agree with the above, I'll create a new ticket. And ignore the
warning in class tomorrow ...
--
Ticket URL: <http://trac.sagemath.org/ticket/14319#comment:48>
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/groups/opt_out.
