#9247: A collection of little improvements to elliptic curves
-------------------------------+--------------------------------------------
Reporter: rlm | Owner: cremona
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-4.5
Component: elliptic curves | Keywords:
Author: Robert Miller | Upstream: N/A
Reviewer: John Cremona | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Comment(by rlm):
Replying to [comment:3 rlm]:
> I'm wondering if by default, E.saturate() shouldn't print all that stuff
from `mwrank`. Oddly, it doesn't show up in doctests, since it goes
straight to the terminal.
More specifically, with the patch here applied to `sage-4.4.4.alpha1` the
following doctest illustrates the printing I'm talking about:
{{{
sage: EllipticCurve([0, 0, 1, -79, 342]).regulator(proof=False) # long
time (seconds)
Saturation index bound = 265
WARNING: saturation at primes p > 100 will not be done;
points may be unsaturated at primes between 100 and index bound
Failed to saturate MW basis at primes [ ]
*** saturation possibly incomplete at primes [ ]
14.7905275701311
}}}
When I do this from the command line, that is exactly what I get. However,
when I run long doctests in `ell_rational_field.py` the following part
appears in the terminal from which I'm running the tests, and is a little
misleading:
{{{
**********************************************************************
File "/Users/rlmill/sage-4.4.4.alpha1/devel/sage-
main/sage/schemes/elliptic_curves/ell_rational_field.py", line 1212:
sage: M(1/7)
Expected:
2
Got:
-2
Saturation index bound = 265
WARNING: saturation at primes p > 100 will not be done;
points may be unsaturated at primes between 100 and index bound
Failed to saturate MW basis at primes [ ]
*** saturation possibly incomplete at primes [ ]
**********************************************************************
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9247#comment:7>
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.
