Merge pull request #92 from brandonwillard/fix-opdef-unify

Fix unification for meta objects without properties

Author: brandonwillard

symbolic_pymc/unify.py

@@ -46,7 +46,7 @@ def wrapper(*args, **kwargs):
 def unify_MetaSymbol(u, v, s):
     if type(u) != type(v):
         return False
-    if hasattr(u, "__all_props__"):
+    if getattr(u, "__all_props__", False):
         s = unify(
             [getattr(u, slot) for slot in u.__all_props__],
             [getattr(v, slot) for slot in v.__all_props__],

tests/tensorflow/test_meta.py

@@ -567,7 +567,6 @@ def test_opdef_func():
     with tf.compat.v1.Session() as sess:
         assert sum_tf.eval() == np.r_[3]
 
-
 @run_in_graph_mode
 def test_tensor_ops():
 

tests/tensorflow/test_unify.py

@@ -23,6 +23,9 @@ def test_operator():
 
     assert add_mt == mt.add
 
+    assert unify(mt.mul, mt.matmul) is False
+    assert unify(mt.mul.op_def, mt.matmul.op_def) is False
+
 
 @run_in_graph_mode
 def test_etuple_term():