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

Infinite matrix groups over QQ fail for is_finite() #8550

Closed
rbeezer mannequin opened this issue Mar 17, 2010 · 10 comments
Closed

Infinite matrix groups over QQ fail for is_finite() #8550

rbeezer mannequin opened this issue Mar 17, 2010 · 10 comments

Comments

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Mar 17, 2010

sage: H=SL(2,QQ)
sage: H.is_finite()
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)

/sage/dev/<ipython console> in <module>()

/sage/dev/local/lib/python2.6/site-packages/sage/groups/matrix_gps/matrix_group.pyc in is_finite(self)
    343         if self.base_ring().is_finite():
    344             return True
--> 345         return self._gap_().IsFinite().bool()
    346
    347     def cardinality(self):

/sage/dev/local/lib/python2.6/site-packages/sage/groups/matrix_gps/matrix_group.pyc in _gap_(self, G)
    217             return SageObject._gap_(self, G)
    218         except TypeError:
--> 219             raise NotImplementedError, "Matrix group over %s not implemented."%self.__R
    220
    221     def __cmp__(self, H):

NotImplementedError: Matrix group over Rational Field not implemented.

GL fails similarly. Other rings (ZZ, finite fields) seem to work OK, so perhaps this is restricted to something peculiar to the rationals?

CC: @videlec

Component: algebra

Author: Frédéric Chapoton

Branch/Commit: 9892a36

Reviewer: Travis Scrimshaw

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

@rbeezer rbeezer mannequin added c: algebra labels Mar 17, 2010
@rbeezer rbeezer mannequin assigned aghitza Mar 17, 2010
@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Sep 23, 2010

comment:1

From the Gap manual, I get the impression that Gap won't allow you to construct a group unless it is finitely generated (and it knows how to calculate a set of generators). That's why Gap will allow you to work with GL(2, ZZ) but not GL(2, QQ).

@fchapoton
Copy link
Contributor

Author: Frédéric Chapoton

@fchapoton
Copy link
Contributor

New commits:

9892a36trac 8550 fix order of SL(2,QQ)

@fchapoton
Copy link
Contributor

Commit: 9892a36

@fchapoton
Copy link
Contributor

Branch: u/chapoton/8550

@fchapoton
Copy link
Contributor

comment:3

green bot, please review

@tscrim
Copy link
Collaborator

tscrim commented Nov 30, 2017

comment:4

LGTM.

@tscrim
Copy link
Collaborator

tscrim commented Nov 30, 2017

Reviewer: Travis Scrimshaw

@fchapoton
Copy link
Contributor

comment:5

thanks

@fchapoton fchapoton added this to the sage-8.2 milestone Dec 1, 2017
@vbraun
Copy link
Member

vbraun commented Dec 14, 2017

Changed branch from u/chapoton/8550 to 9892a36

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

4 participants