#6386: [with new patch, needs review] Implement elliptic exponential
---------------------------+------------------------------------------------
Reporter: cremona | Owner: was
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.1
Component: number theory | Keywords: elliptic curve
Reviewer: | Author: John Cremona
Merged: |
---------------------------+------------------------------------------------
Comment(by robertwb):
Looks good. Using check=False is fine, there's really no way around it.
I understand your point, but I would be more comfortable if for
antilogarithm you at least bounded the magnitude of the imaginary part as
a function of the error in the real part. Also, it has undesirable
behavior when x or y (or both) are supposed to be 0, but are off by
machine epsilon. Neither of these are regressions to what was there
before, and I like the doctests, so I'm not stuck on this.
Positive review pending acceptance of the tiny patch above.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6386#comment:5>
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
-~----------~----~----~----~------~----~------~--~---
