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

coxeter sorting words #18590

Closed
sagetrac-vpilaud mannequin opened this issue Jun 2, 2015 · 24 comments
Closed

coxeter sorting words #18590

sagetrac-vpilaud mannequin opened this issue Jun 2, 2015 · 24 comments

Comments

@sagetrac-vpilaud
Copy link
Mannequin

sagetrac-vpilaud mannequin commented Jun 2, 2015

Coxeter sorting is used to define sortable elements of a Coxeter group.

New methods are:

  • rank of the Coxeter group
  • support of an element (the simple reflections that appear in the reduced expressions of the element)
  • has_full_support tells if the support contains all simple reflections
  • coxeter_sorting_word returns the c-sorting word of an element
  • is_coxeter_sortable tells if an element is c-sortable

CC: @stumpc5 @VivianePons

Component: combinatorics

Keywords: Sage Days 64.5

Author: Vincent Pilaud

Branch/Commit: 6b47229

Reviewer: Viviane Pons, Christian Stump

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

@sagetrac-vpilaud sagetrac-vpilaud mannequin added this to the sage-6.8 milestone Jun 2, 2015
@sagetrac-vpilaud sagetrac-vpilaud mannequin added the p: major / 3 label Jun 2, 2015
@sagetrac-vpilaud
Copy link
Mannequin Author

sagetrac-vpilaud mannequin commented Jun 2, 2015

Branch: u/vpilaud/coxeter_sorting_words

@sagetrac-vpilaud
Copy link
Mannequin Author

sagetrac-vpilaud mannequin commented Jun 2, 2015

Commit: deea7d9

@sagetrac-vpilaud
Copy link
Mannequin Author

sagetrac-vpilaud mannequin commented Jun 2, 2015

Changed keywords from none to Sage Days 64.5

@sagetrac-vpilaud

This comment has been minimized.

@sagetrac-vpilaud
Copy link
Mannequin Author

sagetrac-vpilaud mannequin commented Jun 2, 2015

New commits:

deea7d9add support for sorting words

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

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

40cf52badd rank function and documentation on coxeter sorting words in coxeter_groups

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

Changed commit from deea7d9 to 40cf52b

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

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

d1d24a6fix bug documentation

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 2, 2015

Changed commit from 40cf52b to d1d24a6

@VivianePons
Copy link

@VivianePons
Copy link

Reviewer: Viviane Pons

@VivianePons
Copy link

Author: Vincent Pilaud

@VivianePons
Copy link

Changed commit from d1d24a6 to d1db90c

@VivianePons
Copy link

comment:6

I did some small formating fixes, and I found this error:

sage: W = CoxeterGroup(["A",3])
sage: c = W.from_reduced_word([1,2,3])
sage: l = [w for w in W if w.is_coxeter_sortable(c)]
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-3-ab902ac679a2> in <module>()
----> 1 l = [w for w in W if w.is_coxeter_sortable(c)]

/home/viviane/sage/local/lib/python2.7/site-packages/sage/categories/coxeter_groups.pyc in is_coxeter_sortable(self, c, sorting_word)
   2064                         return False
   2065                 else:
-> 2066                     containment_list[s] = False
   2067                 i += 1
   2068                 if i == n:

IndexError: list assignment index out of range

Also, please update the ticket description to give more details


New commits:

d1d24a6fix bug documentation
d1db90cSome small doc formating fix

@sagetrac-vpilaud
Copy link
Mannequin Author

sagetrac-vpilaud mannequin commented Jun 2, 2015

@sagetrac-vpilaud

This comment has been minimized.

@sagetrac-vpilaud
Copy link
Mannequin Author

sagetrac-vpilaud mannequin commented Jun 2, 2015

New commits:

70315c8fix bug is_sortable_element

@sagetrac-vpilaud
Copy link
Mannequin Author

sagetrac-vpilaud mannequin commented Jun 2, 2015

Changed commit from d1db90c to 70315c8

@VivianePons
Copy link

Changed commit from 70315c8 to 6b47229

@VivianePons
Copy link

comment:9

I've made a few small format fix. I'm happy with the ticket. Christian, if you're happy too you can put it to positive review.


New commits:

6b47229Small formating fixes (adding outputs, fixing doc indent)

@VivianePons
Copy link

@stumpc5
Copy link
Contributor

stumpc5 commented Jun 3, 2015

comment:10

looks good!

@stumpc5
Copy link
Contributor

stumpc5 commented Jun 3, 2015

Changed reviewer from Viviane Pons to Viviane Pons, Christian Stump

@vbraun
Copy link
Member

vbraun commented Jun 4, 2015

Changed branch from u/VivianePons/coxeter_sorting_words to 6b47229

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

3 participants