Skip to content

Tableau.to_permutation equals Tableau.reading_word_permutation and both are broken #14724

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
stumpc5 opened this issue Jun 11, 2013 · 9 comments
Closed

Tableau.to_permutation equals Tableau.reading_word_permutation and both are broken #14724

stumpc5 opened this issue Jun 11, 2013 · 9 comments
Assignees
@sagetrac-sage-combinat
Labels
c: combinatorics p: major / 3 t: bug
Milestone
sage-5.11

Comments

@stumpc5
Copy link
Contributor

stumpc5 commented Jun 11, 2013

The following doesn't work:

sage: T = Tableau([1,1]) 
...
ValueError: The permutation has length 2 but its maximal element is 1. Some element may be repeated, or an element is missing, but there is something wrong with its length.

Depends on #8392

CC: @sagetrac-chrisjamesberg @VivianePons @tscrim

Component: combinatorics

Keywords: permutation, reading word, FindStatDays01

Author: Christian Stump

Reviewer: Travis Scrimshaw

Merged: sage-5.11.beta1

Issue created by migration from https://trac.sagemath.org/ticket/14724
@stumpc5 stumpc5 added this to the sage-5.11 milestone Jun 11, 2013
@stumpc5
Copy link
Contributor Author

stumpc5 commented Jun 12, 2013

Changed keywords from permutation, reading word to permutation, reading word, FindStatDays

@stumpc5
Copy link
Contributor Author

stumpc5 commented Jun 12, 2013

Changed keywords from permutation, reading word, FindStatDays to permutation, reading word, FindStatDays01

@tscrim
Copy link
Collaborator

tscrim commented Jun 12, 2013

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jun 12, 2013

Dependencies: #8392

@stumpc5
Copy link
Contributor Author

stumpc5 commented Jun 12, 2013

comment:4

uploaded the rebased version. Ready for review again!

@tscrim
Copy link
Collaborator

tscrim commented Jun 12, 2013

comment:5

Looks good to me.

@tscrim
Copy link
Collaborator

tscrim commented Jun 12, 2013

comment:6

The deprecated_function_alias does not work properly with the combinatorial_map decorator.

@stumpc5
Copy link
Contributor Author

stumpc5 commented Jun 12, 2013

comment:7

Attachment: trac_14724-tableau_to_permutation_and_friend-cs.patch.gz

Fixed in the new version of the patch - Travis and me verified it...

@jdemeyer
Copy link
Contributor

Merged: sage-5.11.beta1

@a-andre a-andre mentioned this issue Jun 26, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-sage-combinat sagetrac-sage-combinat

Labels
c: combinatorics p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.11
Development

No branches or pull requests
3 participants
@tscrim @stumpc5 @jdemeyer