eja: add a comment about row/column vectors to eja_sn().

[sage.d.git] / mjo / eja / euclidean_jordan_algebra.py

diff --git a/mjo/eja/euclidean_jordan_algebra.py b/mjo/eja/euclidean_jordan_algebra.py
index f71f75d690054a0facd849de639d7a0ead3b8cbe..c47f4003690905278904327e63ddaf7dcf9e0b44 100644 (file)
--- a/mjo/eja/euclidean_jordan_algebra.py
+++ b/mjo/eja/euclidean_jordan_algebra.py
@@ -643,9 +643,13 @@ def eja_sn(dimension, field=QQ):
     W = V.span( mat2vec(s) for s in S )
 
     for s in S:
-        # Brute force the right-multiplication-by-s matrix by looping
+        # Brute force the multiplication-by-s matrix by looping
         # through all elements of the basis and doing the computation
-        # to find out what the corresponding row should be.
+        # to find out what the corresponding row should be. BEWARE:
+        # these multiplication tables won't be symmetric! It therefore
+        # becomes REALLY IMPORTANT that the underlying algebra
+        # constructor uses ROW vectors and not COLUMN vectors. That's
+        # why we're computing rows here and not columns.
         Q_rows = []
         for t in S:
             this_row = mat2vec((s*t + t*s)/2)