#16234: Assorted fixes and optimizations in sage-combinat (mostly partitions and
tableaux)
-------------------------------------+-------------------------------------
       Reporter:  darij              |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.4
      Component:  combinatorics      |   Resolution:
       Keywords:  combinat,          |    Merged in:
  tableaux, partitions, findstat,    |    Reviewers:
  documentation, integer_list,       |  Work issues:
  mutability                         |       Commit:
        Authors:  Darij Grinberg     |  045ada00385e37ac746b7320388d13c2e0d9d5a2
Report Upstream:  N/A                |     Stopgaps:
         Branch:  public/combinat    |
  /tableaux-et-al-doc                |
   Dependencies:                     |
-------------------------------------+-------------------------------------
Changes (by VivianePons):

 * status:  needs_review => needs_work


Comment:

 Hi Darij, thank you very much for this work!


 There is something wrong with the documentation of
 `symmetric_group_action_on_values`. I must say that I didn't know this
 operation, so I don't know its definition. But from the code, it seems
 that your documentation doesn't quite correspond to what is done in the
 code. When you write this:


 {{{
         Then, let `a` be the number of opening
         parentheses in the word, and `b` the number of closing
         parentheses (notice that all opening parentheses are left of
         all closing parentheses). Replace the first `b` parentheses
         by the letters `i+1`, and replace the remaining `a` parentheses
         by the letters `i`.
 }}}


 Indeed, the action of `s_1` on `[[1,2]]` should give `[[2,1]]` which is
 not a semi-standard tableau... (you obtain the unmatched parenthesizes
 `)(`).  The code answers `[[1,2]]`, I must say that it is quite obscure
 and I don't understand what they're doing. But obviously, the
 documentation does not explain it correctly. Do you have the paper
 reference where this operation is described?

 Apart from that, it seems good to me.

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