#19229: Bug in elliptic curve Galois Representation
-------------------------------------+-------------------------------------
       Reporter:  cremona            |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.9
      Component:  elliptic curves    |   Resolution:
       Keywords:  Galois             |    Merged in:
  representations                    |    Reviewers:
        Authors:  John Cremona       |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  u/cremona/19229    |  df3d3f7c5819198be47c0f3dc8d7419066cee34c
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by cremona):

 * status:  new => needs_review


Comment:

 Fixed as follows: on construction of the GaloisRepresentation object the
 CM status of the elliptic curve is assessed and cached.  This enables some
 cleaning of the code later where we never have to rely on the raising of
 exceptions to catch situations which only happen in the CM case.  This
 does not in itself fix the bug, we just get a different error, so the
 second fix is to make the variable name of a number field constructed on
 one function not clash.

 Appropriate doctests added here and in the docstring for
 E.isogeny_class().

--
Ticket URL: <http://trac.sagemath.org/ticket/19229#comment:4>
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.

Reply via email to