]> gitweb.michael.orlitzky.com - sage.d.git/commitdiff
Revert "eja: always use the rationals for denormalized bases."
authorMichael Orlitzky <michael@orlitzky.com>
Thu, 29 Aug 2019 15:45:52 +0000 (11:45 -0400)
committerMichael Orlitzky <michael@orlitzky.com>
Thu, 29 Aug 2019 15:46:16 +0000 (11:46 -0400)
This reverts commit 1e3bfabaac18a1118fc4afd632265d91d3d0ec6c.
It broke the tests, and next time I'll do it one step at a time
to keep them working.

mjo/eja/eja_algebra.py

index 7c2eda211232ceac757a6d6caa40ae6722528349..4673e916433509876638cfaeabfba5883c2a0192 100644 (file)
@@ -1263,7 +1263,7 @@ class ComplexMatrixEuclideanJordanAlgebra(MatrixEuclideanJordanAlgebra):
         if not n.mod(2).is_zero():
             raise ValueError("the matrix 'M' must be a complex embedding")
 
         if not n.mod(2).is_zero():
             raise ValueError("the matrix 'M' must be a complex embedding")
 
-        field = QQ
+        field = M.base_ring() # This should already have sqrt2
         R = PolynomialRing(field, 'z')
         z = R.gen()
         F = NumberField(z**2 + 1,'i', embedding=CLF(-1).sqrt())
         R = PolynomialRing(field, 'z')
         z = R.gen()
         F = NumberField(z**2 + 1,'i', embedding=CLF(-1).sqrt())
@@ -1403,7 +1403,7 @@ class ComplexHermitianEJA(ComplexMatrixEuclideanJordanAlgebra, KnownRankEJA):
             True
 
         """
             True
 
         """
-        R = PolynomialRing(QQ, 'z')
+        R = PolynomialRing(field, 'z')
         z = R.gen()
         F = NumberField(z**2 + 1, 'I', embedding=CLF(-1).sqrt())
         I = F.gen()
         z = R.gen()
         F = NumberField(z**2 + 1, 'I', embedding=CLF(-1).sqrt())
         I = F.gen()