I don't know if we can make a test for that without causing problems. How
do you plan to do that?


On Tue, Apr 15, 2014 at 2:26 PM, Xavier Bouthillier <
notifications@github.com> wrote:

> In pylearn2/utils/mem.py:
>
> > +
> > +    This is to help users understand what could have possibly caused the
> > +    memory error. A more detailed explanation needs to be given for
> > +    each case.
> > +
> > +    Parameters
> > +    ----------
> > +    value: str
> > +        String explaining what the program was trying to do and
> > +        why the memory error possibly happened.
> > +    """
> > +    def __init__(self, value):
> > +        # could add more typical errors to this string
> > +        self.value += ("\n + Make sure you use a 64bit python version. "
> > +                       "32bit python version can only access 2GB of memory on "
> > +                       "windows and 4GB of memory on linux/OSX.")
>
> MemoryError doesn't have an attribute "value". That was a dummy error and
> Travis test should have failed. I'll add a test_mem.py
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/lisa-lab/pylearn2/pull/817/files#r11649665>
> .
>