Merge pull request #1426 from lamblin/small_fixes

Small fixes

Author: bouthilx

doc/internal/pull_request_checklist.txt

@@ -325,3 +325,8 @@ Use the `with` statement, i.e.::
 
 This is cleaner and ensures that the file always gets closed, even in an
 error condition.
+
+Are you adding new files or changing files permissions?
+=======================================================
+The files containing unit tests (named `test_....py`) should never be
+executable, otherwise nose will ignore them, and not execute the tests.

pylearn2/scripts/papers/maxout/compute_test_err.py

@@ -51,8 +51,6 @@ def accs():
                             data_specs = model.cost_from_X_data_specs())
     for item in iterator:
         x_arg, y_arg = item
-        if Xb.ndim > 2:
-            x_arg = test.get_topological_view(x_arg)
         mf1_accs.append(batch_acc(x_arg, y_arg)[0])
     return sum(mf1_accs) / float(len(mf1_accs))