#19561: enforce Sage integers in Krawtchouk
-------------------------+-------------------------------------------------
Reporter: | Owner:
dimpase | Status: needs_review
Type: | Milestone: sage-6.10
defect | Resolution:
Priority: major | Merged in:
Component: | Reviewers:
coding theory | Work issues:
Keywords: | Commit:
Authors: Dima | 2dfdcfed2e78dda3e0b20d4e09f2e0494e98b562
Pasechnik | Stopgaps:
Report Upstream: N/A |
Branch: |
u/dimpase/krawfix |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by ppurka):
Can you apply this patch? Otherwise the rest looks good:
{{{#!diff
diff --git a/src/sage/coding/delsarte_bounds.py
b/src/sage/coding/delsarte_bounds.py
index b4f05b6..cd78e3c 100644
--- a/src/sage/coding/delsarte_bounds.py
+++ b/src/sage/coding/delsarte_bounds.py
@@ -21,7 +21,7 @@ def Krawtchouk(n,q,l,x,check=True):
Compute ``K^{n,q}_l(x)``, the Krawtchouk polynomial.
See :wikipedia:`Kravchuk_polynomials`; It is defined by the
generating function
- `(1-(q-1)z)^{n-x}(1-z)^k=\sum_{l} K^{n,q}_k(x)z^l` and is equal to
+ `(1-(q-1)z)^{n-x}(1-z)^x=\sum_{l} K^{n,q}_l(x)z^l` and is equal to
.. math::
@@ -45,7 +45,7 @@ def Krawtchouk(n,q,l,x,check=True):
TESTS:
- check that the bug reported on #19561 is fixed::
+ check that the bug reported on :trac:`19561` is fixed::
sage: Krawtchouk(3,2,3,3)
-1
}}}
--
Ticket URL: <http://trac.sagemath.org/ticket/19561#comment:12>
Sage <http://www.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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
