Opened 4 years ago
Closed 4 years ago
#22729 closed enhancement (fixed)
Expose some internal variables computed in hecke_series
| Reported by: | mmasdeu | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | sage-8.0 |
| Component: | modular forms | Keywords: | overconvergent, |
| Cc: | davidloeffler | Merged in: | |
| Authors: | Marc Masdeu | Reviewers: | Frédéric Chapoton |
| Report Upstream: | N/A | Work issues: | |
| Branch: | aa3e71f (Commits, GitHub, GitLab) | Commit: | aa3e71fda2926a11883dcf24a568637f2d4f86f9 |
| Dependencies: | Stopgaps: |
Status badges
Description
This ticket exposes (by turning on a flag) certain nontrivial data computed in the hecke_series which is currently hidden from the user. This may be (is) useful in other applications of A.Lauder's code.
Change History (9)
comment:1 Changed 4 years ago by
- Branch set to u/mmasdeu/22729-improve-hecke-series
- Cc davidloeffler added
- Commit set to 7774636bccb7975aca5659acabdd2a20bf332240
- Status changed from new to needs_review
comment:2 Changed 4 years ago by
- Branch changed from u/mmasdeu/22729-improve-hecke-series to public/22729
- Commit changed from 7774636bccb7975aca5659acabdd2a20bf332240 to f42be53b8e0f7f594a077c4e67157ffa653123e8
- Reviewers set to Frédéric Chapoton
comment:3 Changed 4 years ago by
- Commit changed from f42be53b8e0f7f594a077c4e67157ffa653123e8 to aa3e71fda2926a11883dcf24a568637f2d4f86f9
Branch pushed to git repo; I updated commit sha1. New commits:
| aa3e71f | trac 22729 more doc and code details
|
comment:4 Changed 4 years ago by
I made another cosmetic commit, as I had forgotten to do half of the job.
comment:5 Changed 4 years ago by
a question: in the second function
+ the integers ``elldash`` and ``mdash``.
shouldn't elldash be ell ?
comment:6 follow-up: ↓ 7 Changed 4 years ago by
ping ? is there anybody out there ?
comment:7 in reply to: ↑ 6 Changed 4 years ago by
Oops, I think that the email got buried among others...
I still think that it should say what it says. The integer ell can be deduced from the size of the matrices in Alist...
Thank you for looking into this!
Replying to chapoton:
ping ? is there anybody out there ?
comment:9 Changed 4 years ago by
- Branch changed from public/22729 to aa3e71fda2926a11883dcf24a568637f2d4f86f9
- Resolution set to fixed
- Status changed from positive_review to closed
New commits:
Fixed bug when lifting p-automorphic forms of weight > 2.Added doctest to show that bug is fixed.Exposed intermediate data computed in some functions which is useful in other applications.