#18590: coxeter sorting words
-------------------------------------+-------------------------------------
       Reporter:  vpilaud            |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.8
      Component:  combinatorics      |   Resolution:
       Keywords:  Sage Days 64.5     |    Merged in:
        Authors:  Vincent Pilaud     |    Reviewers:  Viviane Pons
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/VivianePons/coxeter_sorting_words|  d1db90cb346dfafac5f6e512ecdb956862828aef
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Vincent Pilaud', 'oldvalue': ''}):

 * status:  needs_review => needs_work
 * author:   => Vincent Pilaud
 * cc: VivianePons (added)
 * branch:  u/vpilaud/coxeter_sorting_words =>
     u/VivianePons/coxeter_sorting_words
 * reviewer:   => Viviane Pons
 * commit:  d1d24a608ff799549a357239614623baf00dc47a =>
     d1db90cb346dfafac5f6e512ecdb956862828aef


Comment:

 I did some small formating fixes, and I found this error:


 {{{
 sage: W = CoxeterGroup(["A",3])
 sage: c = W.from_reduced_word([1,2,3])
 sage: l = [w for w in W if w.is_coxeter_sortable(c)]
 ---------------------------------------------------------------------------
 IndexError                                Traceback (most recent call
 last)
 <ipython-input-3-ab902ac679a2> in <module>()
 ----> 1 l = [w for w in W if w.is_coxeter_sortable(c)]

 /home/viviane/sage/local/lib/python2.7/site-
 packages/sage/categories/coxeter_groups.pyc in is_coxeter_sortable(self,
 c, sorting_word)
    2064                         return False
    2065                 else:
 -> 2066                     containment_list[s] = False
    2067                 i += 1
    2068                 if i == n:

 IndexError: list assignment index out of range
 }}}

 Also, please update the ticket description to give more details
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=d1d24a608ff799549a357239614623baf00dc47a
 d1d24a6]||{{{fix bug documentation}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=d1db90cb346dfafac5f6e512ecdb956862828aef
 d1db90c]||{{{Some small doc formating fix}}}||

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