added "node" argument to ops that already had a connection pattern

theano/tensor/basic.py

@@ -2097,7 +2097,7 @@ class Shape(Op):
     def infer_shape(self, node, in_shapes):
         return [[len(in_shapes[0])]]
 
-    def connection_pattern(self):
+    def connection_pattern(self, node):
         # the grad returns the gradient with respect to the
         # elements of a tensor variable
         # the elements of the tensor variable do not participate
@@ -2193,6 +2193,9 @@ class SpecifyShape(Op):
         assert len(new_shape) == len(xshape)
         return [new_shape]
 
+    def connection_pattern(self, node):
+        return [[True],[False]]
+
     def grad(self, inp, grads):
         x, s = inp
         gz, = grads
@@ -3886,6 +3889,7 @@ class Subtensor(Op):
         else:
             return scal.as_scalar(a)
 
+
     def make_node(self, x, *inputs):
         x = as_tensor_variable(x)
         inputs = tuple(self.my_as_scalar(a) for a in inputs)
@@ -6461,6 +6465,7 @@ class Dot(Op):
             raise
 
     def grad(self, inp, grads):
+
         x, y = inp
         gz, = grads
         if gz.type.ndim == 0:

theano/tests/test_gradient.py

@@ -329,7 +329,7 @@ def test_disconnected_nan():
             return theano.Apply(self, inputs=[x],
                     outputs = [ x.type(), theano.tensor.scalar() ])
 
-        def connection_pattern(self):
+        def connection_pattern(self, node):
             return [[True, False]]
 
         def grad(self, inputs, output_grads):
@@ -360,5 +360,7 @@ def test_disconnected_nan():
     # connection_pattern functionality worked correctly
 
 
+
+
 if __name__ == '__main__':
     unittest.main()