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.

Sign up for GitHub

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

Jump to bottom

Conductor of trivial Dirichlet character has type int #12669

Closed
sagetrac-johanbosman mannequin opened this issue Mar 14, 2012 · 5 comments
Closed

Conductor of trivial Dirichlet character has type int #12669

sagetrac-johanbosman mannequin opened this issue Mar 14, 2012 · 5 comments
Assignees
@craigcitro
Labels
c: modular forms p: major / 3 t: bug
Milestone
sage-5.0

Comments

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Mar 14, 2012 

sage: G.<a, b> = DirichletGroup(20)
sage: type(a.conductor())
<type 'sage.rings.integer.Integer'>
sage: type(G(1).conductor())
<type 'int'>

It should of course return something of type Integer.

Component: modular forms

Keywords: dirichlet characters

Author: Mike Hansen

Reviewer: David Loeffler

Merged: sage-5.0.beta11

Issue created by migration from https://trac.sagemath.org/ticket/12669
@sagetrac-johanbosman sagetrac-johanbosman mannequin added this to the sage-5.0 milestone Mar 14, 2012
@mwhansen
Copy link
Contributor

Attachment: trac_12669.patch.gz

@mwhansen
Copy link
Contributor

Author: Mike Hansen

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Mar 18, 2012

comment:2

This looks fine, and patchbot's happy.

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Mar 18, 2012

Reviewer: David Loeffler

@jdemeyer
Copy link

Merged: sage-5.0.beta11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@craigcitro craigcitro

Labels
c: modular forms p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.0
Development

No branches or pull requests
3 participants
@mwhansen @craigcitro @jdemeyer