#11346: major bug in the conductor function for elliptic curves over number 
fields
-------------------------------+--------------------------------------------
   Reporter:  was              |          Owner:  cremona        
       Type:  defect           |         Status:  positive_review
   Priority:  critical         |      Milestone:  sage-4.7.1     
  Component:  elliptic curves  |       Keywords:                 
Work_issues:                   |       Upstream:  N/A            
   Reviewer:  Robert Bradshaw  |         Author:  William Stein  
     Merged:                   |   Dependencies:                 
-------------------------------+--------------------------------------------
Changes (by newvalueoldvalue):

  * reviewer:  => Robert Bradshaw
  * author:  => William Stein


Comment:

 Additional comment on this and #11347.  I think a better fix would have
 been to add one line in the constructor for {{{EllipticCurveLocalData}}}
 in sage.schemes.elliptic_curves.ell_local_data, namely
 {{{
     E = E.integral_model()
 }}}
 rather than having a lot of separate functions have to remember to do
 this.  The fixes here and at #11347 are fine by themselves, but it remains
 the case that
 {{{
 sage: K.<g> = NumberField(x^2 - x - 1)
 sage: E = EllipticCurve(K,[0,0,0,-1/48,161/864])
 sage: E.local_data()
 }}}
 fails.

 I don't have time to rework this right now, so have not tagged this "needs
 work"...

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11346#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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to