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
Ambient spaces for dual and affine root systems #14085
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
comment:6
The updated patch fixes two minor doctests failures I had missed and that the patchbot reported. I assume that the third failure in interface/expect.py is unrelated. Cheers, |
This comment has been minimized.
This comment has been minimized.
comment:8
New version with quite a few improvements. Unless I missed something, I consider this patch as final. Please review! |
Changed reviewer from Dan Orr, ... to Dan Orr, Anne Schilling |
comment:9
Anne reviewed this patch on the Sage-Combinat queue. The updated patch includes her changes which were cross checked by Nicolas. We also discussed further the math with Mark Shimozono. Positive review! |
Merged: sage-5.8.beta2 |
This ticket implements ambient spaces for dual and affine root
systems. So now all finite and affine crystallographic root systems
have an ambient space, except for relabeled ones (to be handled in a
separate ticket).
Upcoming applications:
the root or weight lattice)
A couple things handled by this patch:
(toward ambient spaces over fields/rings like RR)
CartanType belongs to the appropriate CartanType classes; this
allowed to remove now unnecessary code.
lattice for type BC dual. A Type free approach would be better.
for backward compatibility. This may be changed in a later ticket.
every root lattice realizations using it
Includes the link with (affine/signed) permutation representations.
CC: @sagetrac-sage-combinat @tscrim @sagetrac-mshimo @anneschilling
Component: combinatorics
Author: Nicolas M. Thiéry
Reviewer: Dan Orr, Anne Schilling
Merged: sage-5.8.beta2
Issue created by migration from https://trac.sagemath.org/ticket/14085
The text was updated successfully, but these errors were encountered: