#13951: (non)archimedian_local_height broken for rational points on elliptic 
curves
over Q
---------------------------------------------+------------------------------
       Reporter:  pbruin                     |         Owner:  cremona          
        
           Type:  defect                     |        Status:  needs_review     
        
       Priority:  major                      |     Milestone:  sage-5.10        
        
      Component:  elliptic curves            |    Resolution:                   
        
       Keywords:  local heights              |   Work issues:                   
        
Report Upstream:  N/A                        |     Reviewers:  John Cremona, 
Peter Bruin
        Authors:  Peter Bruin, John Cremona  |     Merged in:                   
        
   Dependencies:  #12509, #13953, #14609     |      Stopgaps:                   
        
---------------------------------------------+------------------------------

Comment (by pbruin):

 I updated trac13951-local_heights_2.patch; height() now only has one
 return statement, and multiplication by the degree, if necessary, is done
 at the very end.  You are right that I should have made a doctest; I added
 the one you suggested.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13951#comment:25>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to