From 28558ed2e67bec59124f1b89c448633097014fd1 Mon Sep 17 00:00:00 2001 From: Joseph Weston <joseph.weston08@gmail.com> Date: Tue, 5 Dec 2017 17:16:14 +0100 Subject: [PATCH] fix check for sympy classes in sympify As of version 1.1.1 Sympy does not include MutableDenseMatrix in 'all_classes', so expressions containing such terms did not trigger the warning in kwant.continuum.sympify when 'locals' are provided. Closes #180. --- kwant/continuum/_common.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/kwant/continuum/_common.py b/kwant/continuum/_common.py index 5fd97c6c..fa244939 100644 --- a/kwant/continuum/_common.py +++ b/kwant/continuum/_common.py @@ -25,6 +25,9 @@ import warnings from .._common import reraise_warnings +# TODO: remove when sympy correctly includes MutableDenseMatrix (lol). +sympy_classes = set(sympy_classes) | {sympy.MutableDenseMatrix} + momentum_operators = sympy.symbols('k_x k_y k_z', commutative=False) position_operators = sympy.symbols('x y z', commutative=False) -- GitLab