#16764: Add CM detection for elliptic curves over number fields
-------------------------------------+-------------------------------------
Reporter: cremona | Owner:
Type: enhancement | Status: closed
Priority: major | Milestone: sage-6.4
Component: elliptic curves | Resolution: fixed
Keywords: complex | Merged in:
multiplication | Reviewers: Marco Streng, Chris
Authors: John Cremona | Wuthrich
Report Upstream: N/A | Work issues:
Branch: | Commit:
4aea367970e8f902845ed4837a6dd9697619a35e| Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by cremona):
Apologies, of course I meant to write "except AttributeError". I do know
the rules.
To make absolutely sure: the field _CMD stores 0 if it has already been
determined that E does not have CM, and in this case the method
cm_discriminant raises an error -- not my choice but for compatibility
with curves over Q. (I would have preferred to return 0 which is easy for
other coed to check).
I'll be happy to review #17076 (though I seem to have forfeited my right
to give any code a trustworthy review with this lapse).
--
Ticket URL: <http://trac.sagemath.org/ticket/16764#comment:28>
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/d/optout.
