Skip to content

Commit

Permalink
matrices: Remove __mathml__
Browse files Browse the repository at this point in the history
This was introduced in 5009295.
There is no need for this to exist, matrices are handled by `print_mathml`.

Furthermore, this method would never have worked, except for matrices containing only other matrices - `self[i, j].__mathml__()` would always raise `AttributeError`.
  • Loading branch information
eric-wieser committed Aug 6, 2020
1 parent 33e0191 commit e18a7aa
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
1 change: 0 additions & 1 deletion bin/coverage_report.py
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ def make_report(
cov = coverage.coverage()
cov.exclude("raise NotImplementedError")
cov.exclude("def canonize") # this should be "@decorated"
cov.exclude("def __mathml__")
if use_cache:
cov.load()
else:
Expand Down
9 changes: 0 additions & 9 deletions sympy/matrices/matrices.py
Original file line number Diff line number Diff line change
Expand Up @@ -776,15 +776,6 @@ def __len__(self):
"""
return self.rows * self.cols

def __mathml__(self):
mml = ""
for i in range(self.rows):
mml += "<matrixrow>"
for j in range(self.cols):
mml += self[i, j].__mathml__()
mml += "</matrixrow>"
return "<matrix>" + mml + "</matrix>"

def _matrix_pow_by_jordan_blocks(self, num):
from sympy.matrices import diag, MutableMatrix
from sympy import binomial
Expand Down

0 comments on commit e18a7aa

Please sign in to comment.