#15300: Weyl and Clifford Algebras
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  tscrim
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.2
      Component:  algebra            |   Resolution:
       Keywords:  days54             |    Merged in:
        Authors:  Travis Scrimshaw   |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  public/algebras/weyl_clifford-15300|  18b9529914faafb25d26bba0435ef06a9914b619
   Dependencies:  #16037             |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by tscrim):

 I think you should do:

 - `if self.base_ring() is R:` in `_element_constructor_` instead of the
 `==`.
 - Move the unshuffle out into `combinat/combinat.py` since it is a general
 (low-level) operation.
 - Is using the `remove_zeros` option the fastest, as opposed to doing the
 `!= R.zero()` check (which probably should be pulled out?)

 Otherwise your changes look good. For the interior product, I'm using the
 definition I gave (the other one should be equivalent to the first one...)
 with `\alpha` as `x`.

 PS - John, I'm still working on the centers.

--
Ticket URL: <http://trac.sagemath.org/ticket/15300#comment:67>
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.

Reply via email to