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

[with negative review] QuadraticFields and ClassGroups #1400

Closed
sagetrac-ljpk mannequin opened this issue Dec 4, 2007 · 6 comments
Closed

[with negative review] QuadraticFields and ClassGroups #1400

sagetrac-ljpk mannequin opened this issue Dec 4, 2007 · 6 comments

Comments

@sagetrac-ljpk
Copy link
Mannequin

sagetrac-ljpk mannequin commented Dec 4, 2007

SAGE can compute the class group of a quadratic field, but it has issues with computing the order of elements within that class group:

QF.<x>=QuadraticField(-39)
CF=QF.class_group()
CF(QF.ideal(1+x)).order()

gives

NotImplementedErrorTraceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/server2/sage_notebook/worksheets/ljpk/0/code/6.py", line 6, in <module>
    CF(QF.ideal(Integer(1)+x)).order()
  File "/home/sage10/", line 1, in <module>
    
  File "element.pyx", line 1190, in sage.structure.element.MultiplicativeGroupElement.order
  File "element.pyx", line 1130, in sage.structure.element.MonoidElement.multiplicative_order
NotImplementedError

Component: number theory

Keywords: editor_craigcitro

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

@sagetrac-mabshoff

This comment has been minimized.

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-2.9 milestone Dec 4, 2007
@aghitza
Copy link

aghitza commented Apr 25, 2008

Attachment: 1400-quadratic_field_order.patch.gz

@aghitza
Copy link

aghitza commented Apr 25, 2008

comment:2

The attached patch adds this functionality for fractional ideals and for their representatives in class groups.

@ncalexan
Copy link
Mannequin

ncalexan mannequin commented Apr 28, 2008

comment:3

This uses an O(n) algorithm to compute orders. Don't we have generic BSGS O(sqrt(n)) that would be better here?

@ncalexan ncalexan mannequin changed the title QuadraticFields and ClassGroups [with negative review] QuadraticFields and ClassGroups Apr 28, 2008
@craigcitro
Copy link
Member

Changed keywords from none to editor_craigcitro

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Aug 27, 2008

comment:5

Fixed via #3913.

Cheers,

Michael

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