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 groups #2217

Closed
wants to merge 1 commit into from
Closed

Conversation

ulthiel
Copy link
Contributor

@ulthiel ulthiel commented Apr 5, 2023

This is an empty draft pull request to indicate that I would really like to bring the package https://github.com/ulthiel/CoxeterGroups.jl for computation in Coxeter groups into OSCAR. This is work done by @joelgibson, @CameronBraunstein, and @schto223. It currently features:

  • Computation in Coxeter groups with arbitrary Coxeter matrix via a recursive algorithm by Casselman. (By "computation" I always mean writing elements in a unique normal form.)
  • Computation in Coxeter groups for a generalized Cartan matrix via minimal roots algorithm. This is at a speed comparable to Magma (which uses the same algorithm basically). I think it can be made to work for general Coxeter matrices as well.
  • Computation in the symmetric group as Coxeter group (generated by the transpositions). Very fast and useful for this special case.
  • Structures for Coxeter matrices and Coxeter systems, especially "identifying" the type.

Not everything is OSCAR ready and perfect but it's basically there and shouldn't lie around any longer. Help integrating this is greatly welcome!

It would be optimal if the current open PRs #2207 and #2115 on Lie theory will be somewhat compatible (in either direction).

@codecov
Copy link

codecov bot commented Apr 5, 2023

Codecov Report

Merging #2217 (cdeaabf) into master (586ddd0) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2217      +/-   ##
==========================================
- Coverage   73.37%   73.36%   -0.01%     
==========================================
  Files         369      369              
  Lines       49804    49804              
==========================================
- Hits        36543    36540       -3     
- Misses      13261    13264       +3     

see 3 files with indirect coverage changes

@fingolfin
Copy link
Member

Closing in favor of PR #2689

@fingolfin fingolfin closed this Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants