#4333: [with patch, needs review] bernoulli_python doesn't work, remove it
---------------------------+------------------------------------------------
 Reporter:  mhampton       |        Owner:  mhampton
     Type:  defect         |       Status:  new     
 Priority:  major          |    Milestone:  sage-3.2
Component:  number theory  |   Resolution:          
 Keywords:  bernoulli      |  
---------------------------+------------------------------------------------
Changes (by mhampton):

  * summary:  bernoulli_python doesn't work => [with patch, needs review]
              bernoulli_python doesn't work, remove it

Comment:

 The patch removes the file bernoulli.py entirely, since it is broken and
 redundant, and removes the corresponding option from the bernoulli
 function in rings/arith.py.

 I also added some long tests.  I noticed, but did not fix, a problem with
 the GAP version of bernoulli, that affects computations of B_{2256} and
 higher:
 {{{
 sage: bernoulli(2256, algorithm = 'gap')

 Traceback (most recent call last):
   File "<stdin>", line 1, in <module>
   File "/Users/mh/sagetest-notebook/worksheets/admin/65/code/18.py", line
 7, in <module>
     bernoulli(_sage_const_2256 , algorithm = \u0027gap\u0027)
   File "/Volumes/D/sage-3.1.3/local/lib/python2.5/site-
 packages/SQLAlchemy-0.4.6-py2.5.egg/", line 1, in <module>

   File "/Volumes/D/sage-3.1.3/local/lib/python2.5/site-
 packages/sage/rings/arith.py", line 212, in bernoulli
     return Rational(x)
   File "rational.pyx", line 189, in sage.rings.rational.Rational.__init__
 (sage/rings/rational.c:4035)
   File "rational.pyx", line 284, in
 sage.rings.rational.Rational.__set_value (sage/rings/rational.c:4742)
   File "/Volumes/D/sage-3.1.3/local/lib/python2.5/site-
 packages/sage/interfaces/expect.py", line 1533, in _rational_
     return sage.rings.all.Rational(repr(self))
   File "rational.pyx", line 189, in sage.rings.rational.Rational.__init__
 (sage/rings/rational.c:4035)
   File "rational.pyx", line 280, in
 sage.rings.rational.Rational.__set_value (sage/rings/rational.c:4708)
 TypeError: unable to convert <<an integer too large to be
 printed>>/14828319870 to a rational

 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4333#comment:2>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to