#18269: A new structure for experimentation on decoding: communication channels
-------------------------+-------------------------------------------------
Reporter: | Owner:
dlucas | Status: needs_work
Type: | Milestone: sage-6.7
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers:
coding theory | Work issues:
Keywords: | Commit:
Authors: | 874deac06298703bc9cd9d8354302830739d6089
Report Upstream: N/A | Stopgaps:
Branch: |
u/dlucas/channels |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by vdelecroix):
`random_error_position` is not an appropriate name. Perhaps `range_sample`
or `sample_range`. The description is not appropriate as well. Should be
something like
{{{
def sample_range(n, k):
"""
Return a subset of [0,...,n-1] of size k with uniform probability.
INPUT:
- ``n``, ``k`` - integers
OUTPUT:
An ordered list.
AUTHORS:
This function is taken from codinglib
(https://bitbucket.org/jsrn/codinglib/)
written by Johan Nielsen.
"""
}}}
In the same function you should also:
- check the input (just do `n = int(n); k = int(k)`)
- rename the variable `error_position` into something more meaningful
And for the line `sage:set_random_seed(10)` there must be a space `sage:
set_random_seed(10)`.
--
Ticket URL: <http://trac.sagemath.org/ticket/18269#comment:6>
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.
