#13065: simon_two_descent not working
-----------------------------------------------------------+----------------
Reporter: alexc | Owner:
cremona
Type: defect | Status:
needs_review
Priority: major | Milestone:
sage-5.4
Component: elliptic curves | Resolution:
Keywords: EllipticCurve simon_two_descent gp pari | Work issues:
Report Upstream: Reported upstream. No feedback yet. | Reviewers:
Authors: Francis Clarke | Merged in:
Dependencies: | Stopgaps:
-----------------------------------------------------------+----------------
Comment (by cremona):
Sorry I took so long to get to this. I applied the patch to 5.4.beta0 and
had no problems. But I have two comments:
First, we should document which version of the simon script the problem
occurs in (when we put a failing case into the doctest). Checking, I find
that rather incredibly the version of ell.gp shopped with Sage now is
dated 25/03/2009. I'm sure that we have newer versions but tey have
failed to be put into Sage since our attempts (most recently by Martin
Raum and me at the Sage Days in Warwick in December 2011) to do that
failed for reasons I will not attempt to recall here.
Secondly, I see that the patch deletes the definition of 'x'. I am very
wary of doing that since there have been issues in the past where
converting strings from pari to Sage required Sage knowning what 'x' is.
For that reason I am doing a full test, and will report back.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13065#comment:9>
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.
