From 2a3d7383e72864ac42a532c6c4250cdc45b4c363 Mon Sep 17 00:00:00 2001 From: Michael Orlitzky Date: Thu, 18 Jul 2019 17:41:14 -0400 Subject: [PATCH] eja: rename matrix() to operator_matrix(). This leaves the dangerous parent class matrix() method there, but whatever. The tests pass. --- mjo/eja/euclidean_jordan_algebra.py | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) 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): -- 2.44.2