From: Michael Orlitzky <michael@orlitzky.com>
Date: Thu, 18 Jul 2019 21:41:14 +0000 (-0400)
Subject: eja: rename matrix() to operator_matrix().
X-Git-Url: https://gitweb.michael.orlitzky.com/?a=commitdiff_plain;h=2a3d7383e72864ac42a532c6c4250cdc45b4c363;p=sage.d.git

eja: rename matrix() to operator_matrix().

This leaves the dangerous parent class matrix() method there, but
whatever. The tests pass.
---

diff --git a/mjo/eja/euclidean_jordan_algebra.py b/mjo/eja/euclidean_jordan_algebra.py
index c0b7787..e2d644a 100644
--- a/mjo/eja/euclidean_jordan_algebra.py
+++ b/mjo/eja/euclidean_jordan_algebra.py
@@ -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):