Opened 10 years ago

Closed 10 years ago

#10438 closed enhancement (duplicate)

[duplicate] Calculate the character of a newform

Reported by: davidloeffler Owned by: craigcitro
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: modular forms Keywords: newform dirichlet character
Cc: Merged in:
Authors: Reviewers: David Loeffler
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by davidloeffler)

Newforms of level Gamma1(N), or GammaH(N) (after #8716), are always eigenforms for the diamond operators (i.e. they have a nebentypus Dirichlet character) but we don't at present have any way of calculating it.

(This ticket is subsumed in #10451).

Attachments (1)

trac_10438-diamond_eigenvalues.patch​ (9.5 KB) - added by davidloeffler 10 years ago.
Patch against 4.6.1.alpha3

Download all attachments as: .zip

Change History (8)

Changed 10 years ago by davidloeffler

Patch against 4.6.1.alpha3

comment:1 Changed 10 years ago by davidloeffler

  • Description modified (diff)
  • Keywords newform dirichlet character added
  • Status changed from new to needs_review

Here's a patch.

comment:2 Changed 10 years ago by davidloeffler

  • Authors set to David Loeffler

comment:3 Changed 10 years ago by davidloeffler

  • Type changed from defect to enhancement

comment:4 Changed 10 years ago by davidloeffler

  • Status changed from needs_review to needs_work

Just realised this is coded in a rather stupid way (generic Hecke modules don't have a diamond_bracket_operator method, and hence shouldn't have a diamond_eigenvalue method either). I will fix this in conjunction with work on #10451.

comment:5 Changed 10 years ago by davidloeffler

  • Description modified (diff)
  • Milestone changed from sage-4.6.2 to sage-duplicate/invalid/wontfix
  • Status changed from needs_work to needs_review
  • Summary changed from Calculate the character of a newform to [duplicate] Calculate the character of a newform

I propose to close this ticket as "duplicate", since the functionality I proposed to add is included in #10451.

comment:6 Changed 10 years ago by davidloeffler

  • Status changed from needs_review to positive_review

I'm setting this to "positive review" as an invitation to the release manager to close it, because all the functionality I planned to add is subsumed in #10451.

Release manager: please close as duplicate, and *do not* merge the patch.

comment:7 Changed 10 years ago by jdemeyer

  • Authors David Loeffler deleted
  • Resolution set to duplicate
  • Reviewers set to David Loeffler
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.