#6870: [with patch, needs review] Bug in binomial
---------------------------------+------------------------------------------
 Reporter:  hgranath             |       Owner:  somebody     
     Type:  defect               |      Status:  new          
 Priority:  minor                |   Milestone:  sage-4.1.2   
Component:  basic arithmetic     |    Keywords:  binomial     
 Reviewer:  Karl-Dieter Crisman  |      Author:  Hakan Granath
   Merged:                       |  
---------------------------------+------------------------------------------

Comment(by mvngu):

 Replying to [comment:20 kcrisman]:
 > Minh, I tried this again in a branch with no binomial changes, and it is
 still there.  I am restoring positive review, and suggest that one looks
 at #6950 and friends for this.
 The doctest failure I got above is due to a 32- vs. 64-bit issue. On a
 32-bit system, it would report 217. But on a 64-bit system, it would
 report 222. These results are consistent for all the machines I have
 tested on. See my doctest reports for Sage 4.1.2.alpha2:

  * 32- and 64-bit [http://groups.google.com/group/sage-
 devel/browse_thread/thread/ec8e2958f394eb5b Ubuntu]
  * 32- and 64-bit [http://groups.google.com/group/sage-
 devel/browse_thread/thread/e61bb57a2637ba2e Mandriva]
  * 32- and 64-bit [http://groups.google.com/group/sage-
 devel/browse_thread/thread/55d756fb80c94780 Debian]
  * 32- and 64-bit [http://groups.google.com/group/sage-
 devel/browse_thread/thread/ff85e2965dc9e59b Fedora],
 [http://groups.google.com/group/sage-
 devel/browse_thread/thread/4ddf1b90690d4cfa Red Hat, CentOS]
  * 32- and 64-bit [http://groups.google.com/group/sage-
 devel/browse_thread/thread/792bb7c3d1f662ef openSUSE]
  * 32- and 64-bit [http://groups.google.com/group/sage-
 devel/browse_thread/thread/954ecbadeb7676a8 Mac OS X 10.5.8]

 In all of the above reports, the doctest in question pass on 64-bit
 platforms, but fail on 32-bit platforms. I have attached the patch
 `trac_6870-bitness-issue.patch` which takes care of this bitness issue. It
 should be applied on top of `trac_6870-final-v2.patch`. If my patch is
 good, then everything is ready to be merged.

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

Reply via email to