Opened 8 years ago
Closed 7 years ago
#17744 closed defect (fixed)
Fix bruhat_graph
Reported by: | tscrim | Owned by: | sage-combinat |
---|---|---|---|
Priority: | major | Milestone: | sage-6.5 |
Component: | combinatorics | Keywords: | weyl group, bruhat graph |
Cc: | sage-combinat, nthiery, aschilling | Merged in: | |
Authors: | Travis Scrimshaw | Reviewers: | Anne Schilling |
Report Upstream: | N/A | Work issues: | |
Branch: | ee45f17 (Commits, GitHub, GitLab) | Commit: | ee45f1795347288062412cf66a9f93d459f69abb |
Dependencies: | Stopgaps: |
Description
We currently have the following behavior (noted by William Solfstra):
sage: W = WeylGroup("A3", prefix="s") sage: W.bruhat_graph(W.unit(), W.long_element()).edges() []
Change History (4)
comment:1 Changed 8 years ago by
- Branch set to public/combinat/bruhat_graph-17744
- Cc nthiery aschilling added
- Commit set to ee45f1795347288062412cf66a9f93d459f69abb
- Status changed from new to needs_review
comment:2 Changed 8 years ago by
- Reviewers set to Anne Schilling
- Status changed from needs_review to positive_review
comment:3 Changed 8 years ago by
Looks good to me! Thanks for fixing this!
comment:4 Changed 7 years ago by
- Branch changed from public/combinat/bruhat_graph-17744 to ee45f1795347288062412cf66a9f93d459f69abb
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
New commits:
Fix bruhat_graph and some modernizing of its documentation.