#18635: Move WeylGroups.ElementMethods.reflection_to_root to
WeylGroups.ParentMethods.reflection_to_positive_root
-----------------------------------------------+------------------------
       Reporter:  stumpc5                      |        Owner:
           Type:  defect                       |       Status:  new
       Priority:  major                        |    Milestone:  sage-6.8
      Component:  categories                   |   Resolution:
       Keywords:  wely groups, coxeter groups  |    Merged in:
        Authors:                               |    Reviewers:
Report Upstream:  N/A                          |  Work issues:
         Branch:                               |       Commit:
   Dependencies:                               |     Stopgaps:
-----------------------------------------------+------------------------

Comment (by stumpc5):

 > In the Weyl groups case, from my viewpoint it is useful to give the
 result in the combination-of-simple-roots form, so I would prefer to keep
 this.

 I also wanted to have it that way, even for complex reflection groups (by
 {{{vector}}} I meant the Sage class, not the mathematical term). The only
 difference there was that I can access the method {{{reflection_to_root}}}
 from the parent, without having to implement an element class.

 > In the parent method there is still the danger that the element is not a
 reflection and the method call "doesn't make sense".

 True, but giving an object additional input that "doesn't make sense" is a
 typical {{{ValueError}}}, while having a method attached to an instance of
 the element class that does not make sense for this element is a more
 fundamental error.

 The reason for asking this question in the first place was that I would
 like to add generic code (#18597) to the {{{FiniteCoxeterGroups}}}
 category which makes use of this method. If it stays an element method, I
 cannot perform that code if I want to call this method from a complex
 reflection group that happens to be a Coxeter group because it lacks this
 method (but, as mentioned above, I can easily add this method to the
 {{{ParentMethods}}}, while it is hard to add to the {{{ElementsMethods}}}.

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

Reply via email to