#4822: [with patch, needs review] Tweak to the error message for EllipticCurve
-----------------------------+----------------------------------------------
Reporter: ljpk | Owner: davidloeffler
Type: enhancement | Status: new
Priority: minor | Milestone: sage-4.1.2
Component: elliptic curves | Keywords:
Reviewer: | Author:
Merged: |
-----------------------------+----------------------------------------------
Comment(by cremona):
The patch deals with this in a more intelligent way. I did not add
doctests since it's hard to do that when the output message depends on
whether or not the extra database is installed. But I did test it before
and after installing the optional database. The message suggests
installing the optional database iff the desired conductor is betweem
10000 and 13000 and the optional db is not already installed; I did not
actually include the command-line to install it though.
The second patch fixes a doctest which otherwise fails when run after
installing the optional database (but is otherwise independent of this
ticket, except that I wrote it so was my fault anyway!)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4822#comment:3>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
