#5673: [with patch, needs work] Enhanced handling of elliptic curve twists
---------------------------+------------------------------------------------
Reporter: cremona | Owner: was
Type: enhancement | Status: new
Priority: minor | Milestone: sage-3.4.2
Component: number theory | Keywords: elliptic curve twist
---------------------------+------------------------------------------------
Comment(by cremona):
OK, the problem was that Chris's original minimal_quadratic_twist()
function gave back the same curve it it already had minimal conductor,
while mine did not, since my function takes the curve with smallest label.
But the padic code could not cope with a non-trivial twist of the same
conductor! (The bad example was '300b2' for which "my" minimal twist is
'300a2' which is its 5-twist.)
Solution: if the minimal twist has the same conductor, I just throw it
away and use the original curve, as Chris's code used to do.
The patch trac_5673_review.patch fixes this.
Sorry I made two separate patches to fix the failures -- I was not sure I
would succeed with both problems.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5673#comment:9>
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
-~----------~----~----~----~------~----~------~--~---
