#9266: bug in global_integral_model for Elliptic Curves over Number Fields
-------------------------------+--------------------------------------------
   Reporter:  wuthrich         |       Owner:  cremona   
       Type:  defect           |      Status:  new       
   Priority:  minor            |   Milestone:  sage-4.4.5
  Component:  elliptic curves  |    Keywords:            
     Author:                   |    Upstream:  N/A       
   Reviewer:                   |      Merged:            
Work_issues:                   |  
-------------------------------+--------------------------------------------
 The following illustrates the bug. It should be easy to fix.

 {{{
 sage: K.<s> = NumberField(x^2-5)
 sage: w = (1+s)/2
 sage: E = EllipticCurve(K,[2,w])
 sage: E.global_integral_model()
 ...sage/schemes/elliptic_curves/ell_number_field.pyc in
 global_integral_model(self)
     377                    ai = [ai[i]/pi**(e*[1,2,3,4,6][i]) for i in
 range(5)]
     378         for z in ai:
 --> 379             assert z.denominator() == 1, "bug in
 global_integral_model: %s" % ai
     380         return EllipticCurve(list(ai))
     381

 TypeError: not all arguments converted during string formatting
 }}}

 So there are two problems. One that the string is not correctly formatted,
 the other that it is raised. The latter, I believe, is just because the
 wrong thing is tested:

 {{{
 sage: w.denominator()
 2
 sage: w.is_integral()
 True
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9266>
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