#19002: Change behavior for vacancy numbers
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  sage-combinat
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.9
      Component:  combinatorics      |   Resolution:
       Keywords:  rigged             |    Merged in:
  configurations                     |    Reviewers:
        Authors:  Travis Scrimshaw   |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  c1eea6f06caa11b6d3a20ed7ca357b523e45b1b2
  public/rigged_configurations/vacancy_numbers-19002|     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by aschilling):

 Replying to [comment:1 tscrim]:
 > I kinda wanted that 19000 ticket number. :P

 Besides the ticket number, the numbering of the rigged partitions seems to
 be off
 {{{
 sage: RC = RiggedConfigurations(['D',8,1],[[8,1],[6,3]])
 sage: H = RC.module_generators
 sage: rc = H[34]
 sage: rc

 (/)

 0[ ]0

 0[ ]0
 0[ ]0

 0[ ]0
 0[ ]0
 0[ ]0

 0[ ]0
 0[ ]0
 0[ ]0
 0[ ]0

 0[ ][ ]0
 0[ ]0
 0[ ]0
 0[ ]0
 0[ ]0

 0[ ][ ]0
 1[ ]1

 0[ ]0
 0[ ]0
 0[ ]0

 sage: rc[8]
 ---------------------------------------------------------------------------
 IndexError                                Traceback (most recent call
 last)
 <ipython-input-28-f31668b6dd44> in <module>()
 ----> 1 rc[Integer(8)]

 /Applications/sage/src/sage/structure/list_clone.pyx in
 sage.structure.list_clone.ClonableArray.__getitem__
 (/Applications/sage/src/build/cythonized/sage/structure/list_clone.c:4586)()
     651         if isinstance(key, slice):
     652             self._list[key.start:key.stop:key.step]
 --> 653         return self._list[key]
     654
     655     def __setitem__(self, int key, value):

 IndexError: list index out of range
 }}}
 I guess the indexing for `a` starts at 0 rather than `1`. But then the
 documentation is misleading since it is not the math labeling. Also,
 wouldn't it be useful to be able to do
 {{{
 rc.vacancy_number(a,i)
 }}}

 Best,

 Anne

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