#4716: [with patch, needs review] Small bug in KodairaSymbol
---------------------------+------------------------------------------------
Reporter: cremona | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.2.2
Component: number theory | Keywords: Elliptic curves
---------------------------+------------------------------------------------
#4412 had a buglet: for Kodaira Class Im the _roman field was not being
set (it should be 1). This is only currently used in the
tamagawa_exponent() function for elliptic curves over number fields.
One-line patch coming up, plus a corresponding doctest.
This was reported by Tobias Nagel:
{{{
sage: E=EllipticCurve('117a3');
sage: E.tamagawa_exponent(13)
---------------------------------------------------------------------------
AttributeError Traceback (most recent call
last)
/home/tobi/test_Sint/<ipython console> in <module>()
/home/tobi/sage/local/lib/python2.5/site-
packages/sage/schemes/elliptic_curves/ell_rational_field.pyc in
tamagawa_exponent(self, p)
2190 return cp
2191 ks = self.kodaira_type(p)
-> 2192 if ks._roman==1 and ks._n%2==0 and ks._starred:
2193 return 2
2194 return 4
AttributeError: 'KodairaSymbol_class' object has no attribute '_roman'
}}}
The patch is based on 3.2.1.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4716>
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
-~----------~----~----~----~------~----~------~--~---
