Fixed bug in "grad" function of ConvOp when dealing with rectangular images or

theano/sandbox/conv.py

@@ -144,12 +144,12 @@ class ConvOp(Op):
             (img, filters) = (newin, newgz)
             (bsize, nkern) = (self.imshp[0], self.nkern)
             imshp = N.hstack((self.bsize, self.imshp[1:]))
-            kshp  = self.outshp[::-1]
+            kshp  = self.outshp
         elif self.out_mode == 'full':
             (img, filters) = (newgz, newin)
             (bsize, nkern) = (self.nkern, self.imshp[0])
             imshp = N.hstack((self.bsize, self.outshp))
-            kshp  = self.imshp[1:][::-1]
+            kshp  = self.imshp[1:]
         else:
             raise NotImplementedError('Only [full,valid] modes are currently supported.')
 

theano/sandbox/test_conv.py

@@ -426,8 +426,8 @@ class TestConvOp(unittest.TestCase):
         nkern = 4
         bsize = 3
         types = ["float32", "float64"]
-        kshps = [(3,3),(5,5)]# TODO: (6,7)show a bug that is not fixed!!
-        imshps = [(1,5,5),(2,8,8)]#,(2,12,12)] #TODO bugged (2,12,10), (3,12,11):
+        kshps = [(5,5), (6,7)]
+        imshps = [(1,5,5), (2,8,8), (3,8,7)]
         modes = ['valid', 'full']
         unroll_batch=[0,1,3]
         unroll_kern=[0,1,4]