#10153: Canonical generator matrices for linear codes and their automorphism 
groups
------------------------------+---------------------------------------------
   Reporter:  tfeulner        |       Owner:  wdj                               
         
       Type:  enhancement     |      Status:  needs_review                      
         
   Priority:  major           |   Milestone:  sage-4.7                          
         
  Component:  coding theory   |    Keywords:  Automorpism group, canonical 
representative
     Author:  Thomas Feulner  |    Upstream:  N/A                               
         
   Reviewer:                  |      Merged:                                    
         
Work_issues:                  |  
------------------------------+---------------------------------------------
Changes (by tfeulner):

  * status:  needs_work => needs_review


Comment:

 The new patch fixes the doctest failures. They were forced by different
 generator matrices of the Hamming Code on different platforms.

 Replying to [comment:7 wdj]:

 > Just a few minor comments on the documentation, which is much better.
 "Canonization of linear codes over finite rings (we are only supporting
 finite fields and \ZZ_4)" The first sentence becomes the title. It would
 be nicer to simply have "Canonization of linear codes over finite rings"
 and add the parenthetical remark later.

 > "elments [A, B] and [A', B'] are multiplied componentwisely." -->
 "elements [A, B] and [A', B'] are multiplied componentwise."

 Changed.

 > You say "Warning: The program ..." a few times. What program?
 `get_automorphism_generators`?

 Changed. Now, the warning is linking to the corresponding functions.

 > You say "See sage.coding.code_can for the interpretation of the group
 elements." a few times. What does this mean exactly? I am looking at the
 documentation of that module. Do you mean read the source code?

 I tried to link the documentation of this method to the documentation of
 the module via :mod:!`sage.coding.code_can`. But this didn`t work. I have
 removed this sentences in the warnings since the OUTPUT sections of the
 methods will explain the group elements, too.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10153#comment:8>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to