#19511: q-ary symmetric channel class for coding theory
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  dlucas                 |       Status:  needs_review
           Type:         |    Milestone:  sage-6.10
  enhancement            |   Resolution:
       Priority:  major  |    Merged in:
      Component:         |    Reviewers:
  coding theory          |  Work issues:
       Keywords:         |       Commit:
        Authors:  David  |  429b2902c76c94d63e0d9ab748fdbe6a5a100219
  Lucas                  |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/dlucas/qsc           |
   Dependencies:         |
-------------------------+-------------------------------------------------
Changes (by dlucas):

 * commit:  ac23bcc4387e05e1cdd5d8efd832381f59b7738f =>
     429b2902c76c94d63e0d9ab748fdbe6a5a100219


Comment:

 Hello,

 Thanks for your remarks!

 I agree with your clarifications, and of course with your fix.

 > Should this channel-intro text really be in the q-ary symmetric channels
 doc?

 Well, I did that in the case one reads only the documentation for a
 channel class, and not the abstract class one. If you think it's useless,
 I'm ok to remove it.

 > Can you please make a test that input space is of the form Sigma^n, and
 that Sigma has random_element

 Done. Note that I wasn't able to find an example of a Sigma which can be
 of the form Sigma^n
 but can not have a `random_element` method...

 >It would be nice with a "probability_of_exactly_t_errors" and a
 "probability_of_at_most_t_errors" method :-) But I don't insist for now,
 the channel is functioning as it is.

 No need to insist, it's done ;) These two methods can be called on any
 instance of the class, with `t` as parameter. Maybe it can be better to
 propose generic methods, which take `epsilon`, `t` and
 `input_space.dimension()` as parameters, so the user can actually look for
 the best epsilon without having to construct the channel... As you prefer.

 Best,

 David
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=e52e9590d7f7175e3237ec85d3c09dec95af4798
 e52e959]||{{{Updated to 7.0}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=9a224321ad758cf5c027de654bbd156f7c7b9bf2
 9a22432]||{{{Added checks on the input}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=429b2902c76c94d63e0d9ab748fdbe6a5a100219
 429b290]||{{{Added methods to estimate the number of errors}}}||

--
Ticket URL: <http://trac.sagemath.org/ticket/19511#comment:9>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to