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

G2 fundamental weights were the negative of what they should be. #2809

Closed
dwbump mannequin opened this issue Apr 5, 2008 · 2 comments
Closed

G2 fundamental weights were the negative of what they should be. #2809

dwbump mannequin opened this issue Apr 5, 2008 · 2 comments

Comments

@dwbump
Copy link
Mannequin

dwbump mannequin commented Apr 5, 2008

In combinat/root_system.py, the fundamental weights for the various root systems are entered by hand. For G2, the fundamental weights were the negatives of what they should be.


diff -r 80b506b8e07c sage/combinat/root_system.py
--- a/sage/combinat/root_system.py Tue Apr 01 19:18:55 2008 -0700
+++ b/sage/combinat/root_system.py Sat Apr 05 08:40:46 2008 -0700
@@ -788,11 +788,11 @@ class AmbientLattice_g(AmbientLattice_ge
         """
         EXAMPLES:
             sage: CartanType(['G',2]).root_system().ambient_lattice().fundamental_weights()
-            [(-1, 0, 1), (-2, 1, 1)]
+            [(1, 0, -1), (2, -1, -1)]
         """
         return [ c0*self._term(0)+c1*self._term(1)+c2*self._term(2) \
                  for [c0,c1,c2] in
-                 [[-1,0,1],[-2,1,1]]]
+                 [[1,0,-1],[2,-1,-1]]]
 
 
 def WeylDim(type, coeffs):

CC: @sagetrac-sage-combinat

Component: combinatorics

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

@dwbump dwbump mannequin added this to the sage-3.0 milestone Apr 5, 2008
@dwbump dwbump mannequin added c: combinatorics labels Apr 5, 2008
@dwbump dwbump mannequin assigned mwhansen Apr 5, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Apr 5, 2008

comment:1

Attachment: g2.patch.gz

This is a dupe of #2808, so I am closing it. Dan: once you open a ticket and you hit "submit" you should remove the "preview" bit from the url before going on. This is a buglet in trac and you aren't the first one who has been bitten by it.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin removed this from the sage-3.0 milestone Apr 5, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Apr 5, 2008
@fchapoton

This comment has been minimized.

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

2 participants