#12418: adding Delsarte bound for codes
----------------------------------+-----------------------------------------
Reporter: dimpase | Owner: wdj
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.11
Component: coding theory | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: | Merged in:
Dependencies: #12533, #13650 | Stopgaps:
----------------------------------+-----------------------------------------
Comment (by dimpase):
Replying to [comment:27 ppurka]:
rebased to Sage 5.10 and fixed the following:
> 1. `Krawtchouk` is missing a doctest.
> 2. There is no need of `\` in `def delsarte_bound_...`
> 3. There are still many trailing whitespaces. If you use vim then you
can try this command `:%s/[ ]\+$//`
> 4. {{{ - ``q`` -- the length of the alphabet}}} -- this should be "the
''size'' of the alphabet"
> Also I think the the options `q` and `d` can be swapped to be in the
order in which they appear
> in the function definition.
I swapped the docstrings instead.
> 5. {{{- ``solver`` -- the LP/ILP solved }}} --- this should be
''solver''.
> What other solvers are present? They should be listed as options in
this
> variable.
> 6. {{{The bound on the size of the F_2-codes}}} --- this should be
''{{{`F_2`}}}''
(and other {{{F_}}})
> 8. The backend should automatically handle "isinteger=True" at least so
that it is functional. Currently I get a very generic Exception (why is
this only "Exception"?). Maybe it is automatically handled in a later
version of sage? I will have to check against 5.6.beta1 and higher:
The PPL backend does not handle ILP. One might want to improve the way it
reports this error, but not on this ticket.
> {{{
> Exception: This backend does not handle integer variables ! Read the doc
!
> }}}
> 11. {{{ Parameter "algorithm" has the same meaning as in
codesize_upper_bound() }}} --- This should be
''{{{:func:`codesize_upper_bound`}}}''.
> 13. According to the patchbot this needs rebasing against some higher
version of 5.6. I have only beta0 here; I will check it against rc1 after
I have compiled that.
Rebased.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12418#comment:29>
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/groups/opt_out.
