]> gitweb.michael.orlitzky.com - sage.d.git/commitdiff
eja: rename matrix() to operator_matrix().
authorMichael Orlitzky <michael@orlitzky.com>
Thu, 18 Jul 2019 21:41:14 +0000 (17:41 -0400)
committerMichael Orlitzky <michael@orlitzky.com>
Mon, 29 Jul 2019 03:19:01 +0000 (23:19 -0400)
This leaves the dangerous parent class matrix() method there, but
whatever. The tests pass.

mjo/eja/euclidean_jordan_algebra.py

index c0b7787a535b7754c446fcb0046c300b21695579..e2d644a903d8eca09a585064f96e31278197e4d6 100644 (file)
@@ -414,7 +414,7 @@ class FiniteDimensionalEuclideanJordanAlgebra(FiniteDimensionalAlgebra):
             return self.span_of_powers().dimension()
 
 
-        def matrix(self):
+        def operator_matrix(self):
             """
             Return the matrix that represents left- (or right-)
             multiplication by this element in the parent algebra.
@@ -480,13 +480,6 @@ class FiniteDimensionalEuclideanJordanAlgebra(FiniteDimensionalAlgebra):
             fda_elt = FiniteDimensionalAlgebraElement(self.parent(), self)
             return fda_elt.matrix().transpose()
 
-        #
-        # The plan is to eventually phase out "matrix()", which sounds
-        # too much like "matrix_representation()", in favor of the more-
-        # accurate "operator_matrix()". But we need to override matrix()
-        # to keep parent class methods happy in the meantime.
-        #
-        operator_matrix = matrix
 
 
         def minimal_polynomial(self):