Merge pull request #13 from nouiz/interactive_debuger_fix

theano/compile/debugmode.py

@@ -1655,10 +1655,10 @@ class _Maker(FunctionMaker): #inheritance buys a few helper functions
             try:
                 theano.config.compute_test_value = "off"
                 optimizer(env)
-            finally:
-                theano.config.compute_test_value = compute_test_value_orig
 
                 theano.compile.function_module.insert_deepcopy(env, inputs, outputs+additional_outputs)
+            finally:
+                theano.config.compute_test_value = compute_test_value_orig
 
             if i:
                 li = env.equivalence_tracker.event_list

theano/compile/function_module.py

@@ -995,8 +995,6 @@ class FunctionMaker(object):
             start_optimizer = time.time()
             optimizer(env)
             end_optimizer = time.time()
-        finally:
-            theano.config.compute_test_value = compute_test_value_orig
 
             opt_time = end_optimizer - start_optimizer
             mode.optimizer_time += opt_time
@@ -1006,6 +1004,8 @@ class FunctionMaker(object):
 
             #Add deep copy to respect the memory interface
             insert_deepcopy(env, inputs, outputs+additional_outputs)
+        finally:
+            theano.config.compute_test_value = compute_test_value_orig
 
         # initialize the linker
         if not hasattr(linker, 'accept'):

theano/gof/tests/test_compute_test_value.py

@@ -372,3 +372,15 @@ class TestComputeTestValue(unittest.TestCase):
 
         finally:
             theano.config.compute_test_value = orig_compute_test_value
+
+    def test_disabled_during_compilation(self):
+        # We test that it is disabled when we include deep copy in the code
+        # This don't test that it is disabled during optimization, but the code do it.
+        orig_compute_test_value = theano.config.compute_test_value
+        try:
+            theano.config.compute_test_value = 'raise'
+            init_Mu1 = theano.shared(numpy.zeros((5,),dtype=config.floatX)).dimshuffle('x',0)
+
+            f = theano.function([], outputs=[init_Mu1])
+        finally:
+            theano.config.compute_test_value = orig_compute_test_value