Opened 8 years ago

Closed 8 years ago

#12669 closed defect (fixed)

Conductor of trivial Dirichlet character has type int

Reported by: johanbosman Owned by: craigcitro
Priority: major Milestone: sage-5.0
Component: modular forms Keywords: dirichlet characters
Cc: Merged in: sage-5.0.beta11
Authors: Mike Hansen Reviewers: David Loeffler
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

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.

Attachments (1)

trac_12669.patch (1.2 KB) - added by mhansen 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by mhansen

comment:1 Changed 8 years ago by mhansen

  • Authors set to Mike Hansen
  • Status changed from new to needs_review

comment:2 Changed 8 years ago by davidloeffler

  • Reviewers set to David Loeffler
  • Status changed from needs_review to positive_review

This looks fine, and patchbot's happy.

comment:3 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.0.beta11
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.