#11215: Various Pairings for Modular Forms
-------------------------+-------------------------------------------------
Reporter: tdupu | Owner: craigcitro
Type: | Status: needs_work
enhancement | Milestone:
Priority: minor | Resolution:
Component: | Merged in:
modular forms | Reviewers: Aly Deines
Keywords: | Work issues:
Authors: | Commit:
Report Upstream: N/A | 565db93a00b34026408e5eae8d306b54b92408e4
Branch: | Stopgaps:
u/chapoton/11215 |
Dependencies: |
-------------------------+-------------------------------------------------
Changes (by aly.deines):
* status: needs_review => needs_work
* reviewer: => Aly Deines
Comment:
It looks good, works, tests pass, but the last line could be broken into
an extra line or two that would make this more readable. If you do that
I'll give you a positive review. Also, could you add comments to explain
what M.a(), M.b() and M.c() are?
--
Ticket URL: <http://trac.sagemath.org/ticket/11215#comment:6>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
