Skip to content
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.

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

Implement left descent and reduced word for colored permutations #25283

Closed
tscrim opened this issue May 3, 2018 · 14 comments
Closed

Implement left descent and reduced word for colored permutations #25283

tscrim opened this issue May 3, 2018 · 14 comments

Comments

@tscrim
Copy link
Collaborator

tscrim commented May 3, 2018

Because they are nice enough groups, we can define an algorithm using left descents (in a certain sense) to compute a reduced word for a colored permutation using the standard generating set.

CC: @sagetrac-sage-combinat @AndrewAtLarge @stumpc5 @fchapoton

Component: combinatorics

Keywords: colored permutations, complex reflection groups

Author: Travis Scrimshaw

Branch/Commit: f9cca1a

Reviewer: Andrew Mathas

Issue created by migration from https://trac.sagemath.org/ticket/25283

@tscrim tscrim added this to the sage-8.3 milestone May 3, 2018
@tscrim
Copy link
Collaborator Author

tscrim commented May 3, 2018

@tscrim
Copy link
Collaborator Author

tscrim commented May 3, 2018

Commit: 39c0121

@tscrim
Copy link
Collaborator Author

tscrim commented May 3, 2018

New commits:

39c0121Implementation of descents and reduced word for colored permutations.

@fchapoton
Copy link
Contributor

comment:2

coverage is not 100%

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 3, 2018

Changed commit from 39c0121 to 55d7f22

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 3, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

55d7f22Adding missing doctest.

@tscrim
Copy link
Collaborator Author

tscrim commented May 3, 2018

comment:4

Forgot to add that doctest in. Now at 100% coverage.

@AndrewMathas
Copy link
Member

Reviewer: Andrew Mathas

@AndrewMathas
Copy link
Member

comment:5

A nice small patch that's easy to review:) I haven't checked the mathematics but the doc tests do this so I think it's good to go.

@vbraun
Copy link
Member

vbraun commented May 14, 2018

comment:6

Merge conflict

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2018

Changed commit from 55d7f22 to f9cca1a

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 14, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

f9cca1aMerge branch 'public/combinat/colored_permutations_reduced_word-25283' of git://trac.sagemath.org/sage into public/combinat/colored_permutations_reduced_word-25283

@tscrim
Copy link
Collaborator Author

tscrim commented May 14, 2018

comment:8

Trivial conflict.

@vbraun
Copy link
Member

vbraun commented May 18, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants