#20970: Gabidulin Codes
-------------------------------------+-------------------------------------
       Reporter:  arpitdm            |        Owner:
           Type:  enhancement        |       Status:  new
       Priority:  major              |    Milestone:  sage-7.3
      Component:  coding theory      |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Arpit Merchant     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/arpitdm/gabidulin_codes          |  ddaef4138cf29d70428a80539d249461226fbc9d
   Dependencies:  #13215             |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by jsrn):

 Noticed the following: In one place, you should be computing `sigma^t(g)`
 for some power `t` and some element `g`, where `sigma` is the Frobenius
 automorphism. You implement this as `pow(sigma(g), t)`, but that's not
 what it means. Rather, it means `sigma( sigma( ... sigma(g)...)`, where
 there is `t` sigmas. In another place, you compute simply `g^(q^t)`
 instead of calling `sigma`, which is also bad style; it should again be
 `sigma` called `t` times on `g`.

--
Ticket URL: <https://trac.sagemath.org/ticket/20970#comment:4>
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