Opened 5 years ago

Closed 5 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) 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 5 years ago by tscrim

  • Branch set to public/combinat/bruhat_graph-17744
  • Cc nthiery aschilling added
  • Commit set to ee45f1795347288062412cf66a9f93d459f69abb
  • Status changed from new to needs_review

New commits:

​ee45f17Fix bruhat_graph and some modernizing of its documentation.

comment:2 Changed 5 years ago by aschilling

  • Reviewers set to Anne Schilling
  • Status changed from needs_review to positive_review

comment:3 Changed 5 years ago by aschilling

Looks good to me! Thanks for fixing this!

comment:4 Changed 5 years ago by vbraun

  • 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.