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

trivial 100x speedup in coding theory #3270

Closed
rlmill mannequin opened this issue May 22, 2008 · 2 comments
Closed

trivial 100x speedup in coding theory #3270

rlmill mannequin opened this issue May 22, 2008 · 2 comments

Comments

@rlmill
Copy link
Mannequin

rlmill mannequin commented May 22, 2008

Was:

sage: C = ExtendedBinaryGolayCode()
sage: time G = C.permutation_automorphism_group()
CPU times: user 2.39 s, sys: 0.58 s, total: 2.97 s
Wall time: 24.32

Now:

sage: C = ExtendedBinaryGolayCode()
sage: time G = C.permutation_automorphism_group()
CPU times: user 0.19 s, sys: 0.04 s, total: 0.23 s
Wall time: 0.24

CC: @wdjoyner

Component: coding theory

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

@rlmill rlmill mannequin added c: coding theory labels May 22, 2008
@rlmill rlmill mannequin self-assigned this May 22, 2008
@mwhansen
Copy link
Contributor

comment:1

Attachment: 3270-bc_default.patch.gz

Looks good to me.

@mwhansen mwhansen changed the title [easy review] trivial 100x speedup in coding theory trivial 100x speedup in coding theory May 22, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented May 22, 2008

comment:2

Merged in Sage 3.0.2.rc0

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed May 22, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-3.0.2 milestone May 22, 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