#10999: Elliptic curve generators from the database lie on the wrong curve
-------------------------------+--------------------------------------------
Reporter: gagansekhon | Owner: cremona
Type: defect | Status: needs_work
Priority: minor | Milestone: sage-4.7
Component: elliptic curves | Keywords: Cremona database, integral
points, gens
Author: John Cremona | Upstream: N/A
Reviewer: Jamie Weigandt | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Changes (by jdemeyer):
* status: positive_review => needs_work
Comment:
Replying to [comment:5 weigandt]:
> The tests pass.
Really? Both the patchbot and my personal testing seems to indicate
doctest problems. Am I missing something?
{{{
**********************************************************************
File "/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.alpha3/devel/sage-
main/doc/en/bordeaux_2008/elliptic_curves.rst", line 139:
sage: E.padic_regulator(5, 10)
Exception raised:
Traceback (most recent call last):
File
"/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.alpha3/local/bin/ncadoctest.py",
line 1231, in run_one_test
self.run_one_example(test, example, filename, compileflags)
File
"/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.alpha3/local/bin/sagedoctest.py",
line 38, in run_one_example
OrigDocTestRunner.run_one_example(self, test, example, filename,
compileflags)
File
"/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.alpha3/local/bin/ncadoctest.py",
line 1172, in run_one_example
compileflags, 1) in test.globs
File "<doctest __main__.example_3[3]>", line 1, in <module>
E.padic_regulator(Integer(5), Integer(10))###line 139:
sage: E.padic_regulator(5, 10)
File
"/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.alpha3/local/lib/python/site-
packages/sage/schemes/elliptic_curves/padics.py", line
269, in padic_regulator
d = self.padic_height_pairing_matrix(p=p, prec=prec,
height=height, check_hypotheses=False)
File
"/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.alpha3/local/lib/python/site-
packages/sage/schemes/elliptic_curves/padics.py", line
344, in padic_height_pairing_matrix
basis = self.gens()
File
"/mnt/usb1/scratch/jdemeyer/merger/sage-4.7.alpha3/local/lib/python/site-
packages/sage/schemes/elliptic_curves/ell_rational_fie
ld.py", line 1802, in gens
return list(self.__gens[proof]) # return copy so not changed
File "parent.pyx", line 738, in
sage.structure.parent.Parent.__getattr__ (sage/structure/parent.c:5752)
File "parent.pyx", line 177, in
sage.structure.parent.raise_attribute_error (sage/structure/parent.c:2724)
AttributeError: 'EllipticCurve_rational_field' object has no attribute
'_EllipticCurve_rational_field__gens'
**********************************************************************
}}}
{{{
The following tests failed:
sage -t -long -force_lib
devel/sage/doc/en/bordeaux_2008/elliptic_curves.rst # 7 doctests failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/padics.py # 23 doctests failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/ell_generic.py # 13 doctests
failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/padic_lseries.py # 2 doctests
failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/ell_point.py # 11 doctests failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/ell_tate_curve.py # 4 doctests
failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/BSD.py # 4 doctests failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/gp_simon.py # 1 doctests failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/ell_number_field.py # 3 doctests
failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/heegner.py # 33 doctests failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/sha_tate.py # 27 doctests failed
sage -t -long -force_lib
devel/sage/sage/schemes/elliptic_curves/ell_rational_field.py # 17
doctests failed
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10999#comment:6>
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.
