# HG changeset patch
# User Olivier Delalleau <delallea@iro>
# Date 1284649183 14400
# Node ID 9583e908c572f1ca26693d32d22a4a99ecb66ae7
# Parent  01163683c71b56ab5df060974f3f21165c513bff
coding_style: Add a pylearn.compat module?

diff -r 01163683c71b -r 9583e908c572 doc/v2_planning/coding_style.txt
--- a/doc/v2_planning/coding_style.txt	Thu Sep 16 09:26:31 2010 -0400
+++ b/doc/v2_planning/coding_style.txt	Thu Sep 16 10:59:43 2010 -0400
@@ -386,6 +386,8 @@
       in Python 2.4).
       You can use numpy.{all,any} instead of import theano.gof.python25 that 
       define all and any.
+      OD: I think we should have something like pylearn.compat.{all,any}.
+          numpy.{all,any} are meant to be used on arrays only.
 
     * Do not use the `hashlib` module (not supported in Python 2.4).
       You can do as in theano.gof.cc:
@@ -398,6 +400,8 @@
 	    import md5
 	    def hash_from_code(msg):
 	        return md5.new(msg).hexdigest()
+    OD: Yep, we could probably come up with such a wrapper in a pylearn.compat
+        module.
 
 Mercurial commits
 -----------------