Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Permutation groups .degree() function misleads. #3404

Closed
jasongrout opened this issue Jun 12, 2008 · 3 comments
Closed

Permutation groups .degree() function misleads. #3404

jasongrout opened this issue Jun 12, 2008 · 3 comments

Comments

@jasongrout
Copy link
Member

Both of the graphs below are the path graph on 3 vertices. The problem is that the first labeling returns a permutation group of degree 2, when it should be of degree 3.

sage: g=Graph('Bo')
sage: print g.automorphism_group().degree()
2
sage: h=Graph('Bg')
sage: print h.automorphism_group().degree()
3
sage: g.is_isomorphic(h)
True

Component: basic arithmetic

Issue created by migration from https://trac.sagemath.org/ticket/3404

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 12, 2008

comment:1

Unfortunately, since permutation groups are (currently) simply wrappers for GAP permutation groups, the object itself carries no notion of what it acts on, only a cycle representation of its generators.

sage: g=Graph('Bo')
sage: G=g.automorphism_group()
sage: G.degree?
Type:		instancemethod
Base Class:	<type 'instancemethod'>
String Form:	<bound method PermutationGroup_generic.degree of Permutation Group with generators [(1,2)]>
Namespace:	Interactive
File:		/Users/rlmill/sage-3.0.2/local/lib/python2.5/site-packages/sage/groups/perm_gps/permgroup.py
Definition:	G.degree(self)
Docstring:
    
            Synonym for largest_moved_point().
    
            EXAMPLES:
                sage: G = PermutationGroup([[(1,2,3),(4,5)],[(3,4)]])
                sage: G.degree()
                5

However, you can do:

sage: g.automorphism_group(translation=True, orbits=True)
(Permutation Group with generators [(1,2)], {0: 3, 1: 1, 2: 2}, [[0], [1, 2]])

The second is telling you which vertex of the graph is thought of as which integer by the group (here GAP thinks of this as 3 fixed, 4 fixed, 5 fixed, ...) and the third are the actual orbits of all the vertices.

This is a problem in permutation groups, not graphs.

@rlmill rlmill mannequin changed the title graph.automorphism_group returns permutations with the wrong number of elements. [invalid] graph.automorphism_group returns permutations with the wrong number of elements. Jun 12, 2008
@rlmill rlmill mannequin changed the title [invalid] graph.automorphism_group returns permutations with the wrong number of elements. Permutation groups .degree() function misleads. Jun 12, 2008
@mwhansen

This comment has been minimized.

@mwhansen
Copy link
Contributor

mwhansen commented Jun 4, 2009

comment:3

This has been fixed.

----------------------------------------------------------------------
| Sage Version 4.0.1.rc1, Release Date: 2009-06-04                   |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: sage: g=Graph('Bo')
sage: sage: print g.automorphism_group().degree()
3
sage: g.automorphism_group()
Permutation Group with generators [(1,2)]

@mwhansen mwhansen removed this from the sage-4.0.1 milestone Jun 4, 2009
@mwhansen mwhansen closed this as completed Jun 4, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants