#4897: [with patch, needs review] integral_points() misses some points
--------------------------------------------+-------------------------------
Reporter: cremona | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.4
Component: number theory | Resolution:
Keywords: elliptic curve integral points |
--------------------------------------------+-------------------------------
Changes (by cremona):
* summary: integral_points() misses some points => [with patch, needs
review] integral_points() misses some points
Comment:
The patch is based on 3.2.2 + #4901 (which has a positive review but is
not yet merged).
It does what was promised above, i.e. corrects the function
point_preprocessing(). A doctest is added with example from Francois
Glineur. In addition, I reran every single curve in the database (as
reported to sage-nt) and have uploaded the revised files to my web page:
in about 1% of cases (over 8000 curves) there are more integral points
than before.
The only file touched here is
schemes/elliptic_curves/ell_rational_field.py, which is one of the ones
which has already been sphinxified (see #4926) so this patch will need to
be merged with that one at some point.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4897#comment:3>
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
-~----------~----~----~----~------~----~------~--~---
