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

Implementation of the Coxeter-Knuth graph in type A #16845

Closed
anneschilling opened this issue Aug 17, 2014 · 21 comments
Closed

Implementation of the Coxeter-Knuth graph in type A #16845

anneschilling opened this issue Aug 17, 2014 · 21 comments

Comments

@anneschilling
Copy link

This branch implements the Coxeter-Knuth graph of type A. The Coxeter-Knuth relations in this case are given by a a+1 a \sim a+1 a a+1, abc \sim acb if b<a<c and abc \sim bac if a<c<b.

CC: @sagetrac-sage-combinat

Component: combinatorics

Keywords: coxeter group, days60

Author: Anne Schilling

Branch: 44e08e3

Reviewer: Nicolas M. Thiéry

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

@nthiery
Copy link
Contributor

nthiery commented Aug 17, 2014

Changed keywords from coxeter group to coxeter group, days60

@anneschilling
Copy link
Author

@anneschilling
Copy link
Author

New commits:

07c6b2aimplementation of the Coxeter-Knuth graph in type A

@anneschilling
Copy link
Author

@anneschilling
Copy link
Author

Commit: 07c6b2a

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 19, 2014

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

4529af8moved code to finite_coxeter_groups and factored out the computation of the neighbors in type A

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 19, 2014

Changed commit from 07c6b2a to 4529af8

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 19, 2014

Changed commit from 4529af8 to 4d955dc

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 19, 2014

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

4d955dcchanged according to reviewer

@nthiery
Copy link
Contributor

nthiery commented Aug 19, 2014

comment:7

I checked this with Anne. All good to go!


New commits:

4d955dcchanged according to reviewer

New commits:

4d955dcchanged according to reviewer

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Aug 19, 2014

comment:8

It looks like you import dot2tex for nothing.

Nathann

@anneschilling
Copy link
Author

comment:9

Replying to @nathanncohen:

It looks like you import dot2tex for nothing.

Oh yes! Thanks for catching this! I had latex options in the code, but Nicolas made me remove those and I forgot to remove the import. I will fix this tomorrow (as my internet right now does not allow for shh connections).

@vbraun
Copy link
Member

vbraun commented Aug 19, 2014

comment:10

Reviewer name

@vbraun
Copy link
Member

vbraun commented Aug 19, 2014

comment:11

Reviewer name

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 20, 2014

Changed commit from 4d955dc to 44e08e3

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 20, 2014

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

44e08e3removed superfluous import

@anneschilling
Copy link
Author

Reviewer: Nicolas M. Thiery

@anneschilling
Copy link
Author

comment:14

I removed the superfluous import and set Nicolas as a reviewer.

@vbraun
Copy link
Member

vbraun commented Aug 20, 2014

Changed branch from u/aschilling/combinat/16845/coxeter-knuth-graph to 44e08e3

@kcrisman
Copy link
Member

Changed commit from 44e08e3 to none

@kcrisman
Copy link
Member

Changed reviewer from Nicolas M. Thiery to Nicolas M. Thiéry

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