#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:22 kcrisman]:
> I believe you, though I have no way of testing this, as I don't plan to
build a 64-bit Sage any time soon. My point is that, as far as I can
tell, it should be the subject of its own ticket, not this one.
>
> In any case, I an unable to review that part of your patch. I'm sorry
:(
What you can do is get the patch `trac_6870-final-v2.patch` and remove the
hunk:
{{{
1011 1011 sage: B.nvariables()
1012 1012 9
1013 1013 sage: B.nonlinearity()
1014 222
1014 217
1015 1015 """
1016 1016 from sage.misc.randstate import current_randstate
1017 1017 r = current_randstate().python_random()
}}}
from that patch. The new patch would be the same as the original, only
with changes to the file `sage/rings/arith.py`. As for my patch, you could
open another ticket and put the patch there. That way, the patch won't be
lost to history, and you could still review Hakan's changes to
`sage/rings/arith.py`. As for the doctest failure in
`sage/crypto/boolean_function.pyx`, you reference the new ticket from this
ticket. How does that sound?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6870#comment:23>
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
-~----------~----~----~----~------~----~------~--~---
