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

Investigate long time graph_isom case #1360

Closed
rlmill mannequin opened this issue Dec 2, 2007 · 5 comments
Closed

Investigate long time graph_isom case #1360

rlmill mannequin opened this issue Dec 2, 2007 · 5 comments

Comments

@rlmill
Copy link
Mannequin

rlmill mannequin commented Dec 2, 2007

The full discussion is here.

http://groups.google.com/group/sage-support/browse_thread/thread/4c615950a190e3f3

Component: graph theory

Keywords: isomorphism, canonical labeling

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

@rlmill rlmill mannequin added this to the sage-2.9 milestone Dec 2, 2007
@rlmill rlmill mannequin added c: combinatorics labels Dec 2, 2007
@rlmill rlmill mannequin assigned mwhansen Dec 2, 2007
@rlmill
Copy link
Mannequin Author

rlmill mannequin commented Dec 2, 2007

Changed keywords from none to graph isomorphism

@rlmill rlmill mannequin added the c: graph theory label Dec 2, 2007
@rlmill rlmill mannequin assigned rlmill and unassigned mwhansen Dec 2, 2007
@rlmill rlmill mannequin modified the milestones: sage-2.9, sage-2.10 Dec 2, 2007
@rlmill
Copy link
Mannequin Author

rlmill mannequin commented Dec 17, 2007

Changed keywords from graph isomorphism to isomorphism, canonical labeling

@rlmill rlmill mannequin removed the c: combinatorics label Dec 17, 2007
@rlmill
Copy link
Mannequin Author

rlmill mannequin commented Feb 18, 2008

comment:4

After all of the patches that lead to the fix in #1961, this case no longer takes any real time. Once #1961 is merged, this ticket should also be closed!

@rlmill
Copy link
Mannequin Author

rlmill mannequin commented Feb 18, 2008

comment:5

In fact:

sage: time GAut = G.automorphism_group(partition=Pi)
CPU times: user 0.10 s, sys: 0.04 s, total: 0.14 s
Wall time: 0.15

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Feb 19, 2008

comment:6

I ran the computation with #1961 applied and rlm confirms that the result is as expected:

sage: GAut = G.automorphism_group(partition=Pi)
sage: GAut
Permutation Group with generators [(1,6)(9,14)(18,28)(19,29)(34,44)(35,45)(50,60)(51,61)(66,76)(67,77), 
(6,7)(8,9)(28,30)(29,31)(32,34)(33,35)(60,62)(61,63)(64,66)(65,67), 
(5,8)(7,10)(26,32)(27,33)(30,36)(31,37)(58,64)(59,65)(62,68)(63,69), (4,5)(10,11)(24,26)(25,27)(36,38)(37,39)(56,58)(57,59)(68,70)(69,71), 
(3,4)(11,12)(22,24)(23,25)(38,40)(39,41)(54,56)(55,57)(70,72)(71,73),
(2,3)(12,13)(20,22)(21,23)(40,42)(41,43)(52,54)(53,55)(72,74)(73,75)]

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Feb 19, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-2.10.3, sage-2.10.2 Feb 20, 2008
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

1 participant