#1847: [with patch, positive review, needs doctest fix] add nice print method
for
Sha(Elliptic curve)
---------------------------+------------------------------------------------
Reporter: was | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.4.1
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Changes (by mabshoff):
* summary: [with patch, positive review] add nice print method for
Sha(Elliptic curve) [trivial to implement --
requires taste] => [with patch, positive
review, needs doctest fix] add nice print
method for Sha(Elliptic curve)
Comment:
This patch causes the following doctest failure:
{{{
mabsh...@geom:/scratch/mabshoff/sage-3.3.alpha3$ ./sage -t
devel/sage/sage/schemes/elliptic_curves/ell_rational_field.py
sage -t "devel/sage/sage/schemes/elliptic_curves/ell_rational_field.py"
**********************************************************************
File
"/scratch/mabshoff/sage-3.3.alpha3/devel/sage/sage/schemes/elliptic_curves/ell_rational_field.py",
line 3639:
sage: S
Expected:
<class 'sage.schemes.elliptic_curves.sha_tate.Sha'>
Got:
Shafarevich-Tate group for the Elliptic Curve defined by y^2 + y = x^3
- x over Rational Field
**********************************************************************
}}}
I guess the obvious fix is to change the doctest.
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/1847#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
-~----------~----~----~----~------~----~------~--~---
