That is good. I was thinking allocation enough memory to cause a memory
error... Your test it good.


On Tue, Apr 15, 2014 at 2:36 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.")
>
> Just a dummy test that raises a TypicalMemoryError and catches only
> TypicalMemoryError.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/lisa-lab/pylearn2/pull/817/files#r11650245>
> .
>