#7983: Notion of descent/major index in tableau.py is not mathematically
standard
-----------------------------+----------------------------------------------
Reporter: jbandlow | Owner: sage-combinat
Type: defect | Status: needs_work
Priority: major | Milestone:
Component: combinatorics | Keywords:
Author: | Upstream: N/A
Reviewer: jbandlow | Merged:
Work_issues: |
-----------------------------+----------------------------------------------
Changes (by jbandlow):
* status: needs_review => needs_work
Comment:
Thanks for the patch! While I don't have time for a full review now, the
main issue with this patch is going to be backward compatibility. I'm
pretty sure that the Macdonald polynomial code uses these functions, so
sage --testall will probably fail after applying your patch. That part
will not be too hard to fix, but the bigger problem is for people who have
sage code on their own machine. When people upgrade sage and this change
is included (without them necessarily knowing about it) this change could
make their code behave in slightly wrong ways that are not obvious. We
really try to avoid that.
So I think the thing to do is to deprecate 'descents' and 'major_index'
(look up deprecation in the developers guide), use 'i_descents' and
'i_major_index' for the existing statistics (as you have done) and give
the classical statistics some new name. (Suggestions welcome!)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7983#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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
