#13606: Adding anticommutator feature
----------------------------------+-----------------------------------------
       Reporter:  dox             |         Owner:  jason, was  
           Type:  enhancement     |        Status:  needs_review
       Priority:  minor           |     Milestone:  sage-5.7    
      Component:  linear algebra  |    Resolution:              
       Keywords:                  |   Work issues:              
Report Upstream:  N/A             |     Reviewers:              
        Authors:                  |     Merged in:              
   Dependencies:                  |      Stopgaps:              
----------------------------------+-----------------------------------------

Comment (by knsam):

 I'd like to review this ticket. Here are my comments:

 1. Please fix the docstring: we need "anticommutator" and NOT
 "commutator". I'd be glad if you rewrote the docstring along the lines of:

 {{{
 """
 Return the anticommutator of ``self`` and ``other``.

 The anticommutator of two square matrices `A` and `B` of same
 dimension is defined to be `AB + BA`.

 EXAMPLES::


 """
 }}}
 2. Perhaps, we could add atleast one more example, like the trace of
 anticommutator. But, this is just my feeling.

 Other than these, the patch looks great! :)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13606#comment:2>
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].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to