#5113: [with patch, positive review] elliptic curve construction from 
weierstrass
equation
---------------------------+------------------------------------------------
 Reporter:  robertwb       |        Owner:  was     
     Type:  enhancement    |       Status:  new     
 Priority:  major          |    Milestone:  sage-3.3
Component:  number theory  |   Resolution:          
 Keywords:                 |  
---------------------------+------------------------------------------------
Changes (by rlm):

  * summary:  [with patch, needs review] elliptic curve construction from
              weierstrass equation => [with patch, positive
              review] elliptic curve construction from
              weierstrass equation

Comment:

 I'm curious why you don't just do
 {{{
 a1 = -1*f.coefficient(x*y)
 a2 = f.coefficient(x**2)
 }}}
 instead of iterating through `f`. I'm sure it doesn't matter.

 The patch looks good though, positive review.

 I've also fixed printing of elliptic curves, see #5118.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5113#comment:1>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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