#1234: [with patch; with positive review] analytic_rank crashes
--------------------------------+-------------------------------------------
Reporter: zimmerma | Owner: was
Type: defect | Status: new
Priority: minor | Milestone: sage-3.4.1
Component: algebraic geometry | Resolution:
Keywords: |
--------------------------------+-------------------------------------------
Changes (by cremona):
* summary: [with patch; needs review] analytic_rank crashes => [with
patch; with positive review] analytic_rank
crashes
Comment:
Review: excellent and well done. Applies cleanly to 3.3.alpha0 and all
tests in elliptic_curves pass.
Suggestion: I see that for EllipticCurve([1234567,89101112]) which has
conductor 61928339435779485920, sympow and rubinstein know they are beaten
and quit, while cremona just takes a long time (I did not wait). It might
be a good idea to work out a sensible conductor limit for my gp script and
have sage only call the script if under that (perhaps with a parameter to
allow users to override it). But that would depend on many factors (speed
of machine etc), and this patch should not be delayed because of that.
Pass!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/1234#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
-~----------~----~----~----~------~----~------~--~---
