#20564: KleshchevPartitions
-------------------------------------+-------------------------------------
       Reporter:  andrew.mathas      |        Owner:  andrew.mathas
           Type:  enhancement        |       Status:  new
       Priority:  major              |    Milestone:  sage-7.2
      Component:  combinatorics      |   Resolution:
       Keywords:  Kleshchev          |    Merged in:
  partition tuples                   |    Reviewers:
        Authors:  Andrew Mathas      |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  1a15906ffaac3f7fa11c7261cae8e14d3e4ec01d
  u/andrew.mathas/kleshchevpartitions|     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by andrew.mathas):

 Replying to [comment:9 tscrim]:
 > There are also duplicates in the `"down"` iteration. For this, I think
 we should just use the crystal iteration from #15584, which seems to
 match. Do you agree?

 Great. I noticed that I had too many tuples yesterday but didn't have time
 to investigate. I thought that I just needed to add the direction to the
 calls to the good nodes routines in order to get the labelling that you
 are using but, in addition, I should have changed `cell<=c` to `cell>=c`.
 With this fix to the code our labellings agree. I also separated the four
 different iterations (for up/down and level 1/higher levels) into their
 own methods.

 With these changes in place the timing for my code are:
 {{{
 sage: %timeit KleshchevPartitions(5,[3,2,1],3)[:]
 100 loops, best of 3: 6.44 ms per loop
 sage: %timeit KleshchevPartitions(5,[3,2,1],3, direction="down")[:]
 100 loops, best of 3: 6.62 ms per loop
 }}}
 Mind you I am not sure how meaningful this is as there is probably some
 caching going on with `UniqueRepresentation`...

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

Reply via email to