add detail to comment and coding style change.

theano/tensor/nnet/Conv3D.py

@@ -51,10 +51,10 @@ class Conv3D(theano.Op):
 
     def make_node(self, V, W, b, d):
         """
-            :param V: Visible unit, input
-            :param W: Weights, filter
+            :param V: Visible unit, input(batch,row,column,time,in channel)
+            :param W: Weights, filter(out channel,row,column,time,in channel)
             :param b: bias, shape == (W.shape[0],)
-            :param d: strides when moving the filter over the input
+            :param d: strides when moving the filter over the input(dx,dy,dt)
         """
 
         V_ = T.as_tensor_variable(V)
@@ -82,22 +82,22 @@ class Conv3D(theano.Op):
         dCdb = T.sum(dCdH, axis=(0,1,2,3))
         dCdd = None #not differentiable, since d is not continuous
 
-        if 'name' in dir(dCdH) and dCdH.name != None:
+        if 'name' in dir(dCdH) and dCdH.name is not None:
             dCdH_name = dCdH.name
         else:
             dCdH_name = 'anon'
 
-        if 'name' in dir(V) and V.name != None:
+        if 'name' in dir(V) and V.name is not None:
             V_name = V.name
         else:
             V_name = 'anon'
 
-        if 'name' in dir(W) and W.name != None:
+        if 'name' in dir(W) and W.name is not None:
             W_name = W.name
         else:
             W_name = 'anon'
 
-        if 'name' in dir(b) and b.name != None:
+        if 'name' in dir(b) and b.name is not None:
             b_name = b.name
         else:
             b_name = 'anon'

theano/tensor/nnet/ConvTransp3D.py

@@ -50,22 +50,22 @@ class ConvTransp3D(theano.Op):
         dCdRShape = None #not differentiable, since RShape is not continuous
 
 
-        if 'name' in dir(dCdR) and dCdR.name != None:
+        if 'name' in dir(dCdR) and dCdR.name is not None:
             dCdR_name = dCdR.name
         else:
             dCdR_name = 'anon'
 
-        if 'name' in dir(H) and H.name != None:
+        if 'name' in dir(H) and H.name is not None:
             H_name = H.name
         else:
             H_name = 'anon'
 
-        if 'name' in dir(W) and W.name != None:
+        if 'name' in dir(W) and W.name is not None:
             W_name = W.name
         else:
             W_name = 'anon'
 
-        if 'name' in dir(b) and b.name != None:
+        if 'name' in dir(b) and b.name is not None:
             b_name = b.name
         else:
             b_name = 'anon'
@@ -79,9 +79,9 @@ class ConvTransp3D(theano.Op):
 
 
     def perform(self, node, inputs, output_storage):
-		W, b, d, H, RShape = inputs
-		print "\t\t\t\tConvTransp3D python code"
-		output_storage[0][0] = computeR(W,b,d,H,RShape)
+        W, b, d, H, RShape = inputs
+        print "\t\t\t\tConvTransp3D python code"
+        output_storage[0][0] = computeR(W,b,d,H,RShape)
 
     def c_code(self, node, nodename, (W, b, d, H, RShape), outputs, sub):
         fail = sub['fail']
@@ -360,7 +360,7 @@ def computeR(W,b,d,H,Rshape = None):
 		videoWidth = (outputWidth-1) * dc + filterWidth
 		videoDur = (outputDur-1) * dt + filterDur
 
-		if Rshape != None and Rshape[0] != -1:
+		if Rshape is not None and Rshape[0] != -1:
 			if Rshape[0] < videoHeight:
 				print (Rshape[0], videoHeight)
 				assert False