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

Behaviour of the order function for infinite groups #1170

Closed
sagetrac-ljpk mannequin opened this issue Nov 14, 2007 · 4 comments
Closed

Behaviour of the order function for infinite groups #1170

sagetrac-ljpk mannequin opened this issue Nov 14, 2007 · 4 comments

Comments

@sagetrac-ljpk
Copy link
Mannequin

sagetrac-ljpk mannequin commented Nov 14, 2007

When one tries to use the order function on group elements of infinite order, one gets an error:

gl=GL(2,ZZ)
g=gl.gens()[2]
g.order()

In MAGMA, one (often) gets the answer 0 if one calls the Order function on elements of infinite order.

Component: linear algebra

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

@sagetrac-ljpk sagetrac-ljpk mannequin added this to the sage-3.1.2 milestone Nov 14, 2007
@JohnCremona
Copy link
Member

Attachment: 1170-matrix-group-order.patch.gz

@JohnCremona
Copy link
Member

comment:1

The patch fixes this: for consistency with other groups, +Infinity is returned as the order for group elements of infinite order. A doctest has been added.

The patch should apply to 3.1.2.alpha4 and later, and all doctests in sage.groups pass.

Review, Lloyd?

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Sep 4, 2008

comment:3

One small nitpick which I corrected in the patch I applied: The # in the doctests need to be escaped, i.e.

See trac \#1170

I am not sure if this applies in case the docstring is not raw, but let's do it so that downroad we do not get bitten by it.

Cheers,

Michael

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Sep 4, 2008

comment:4

Merged in Sage 3.1.2.rc0

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Sep 4, 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

3 participants