My bad, I did not see the length-checking part of the statement.

> On Feb 21, 2014, at 10:59, mkg <notifications@github.com> wrote:
> 
> In pylearn2/space/__init__.py:
> 
> > @@ -884,9 +1244,9 @@ def __eq__(self, other):
> >          """
> >          return (type(self) == type(other) and
> >                  len(self.components) == len(other.components) and
> > -                all([my_component == other_component for
> > -                     my_component, other_component in
> > -                     zip(self.components, other.components)]))
> > +                all(my_component == other_component for
> > +                    my_component, other_component in
> > +                    zip(self.components, other.components)))
> I actually don't want it to throw an error if the two spaces have different components; I want it to return False. This is why I check for len(self.components) == len(other.components) in the preceding line.
> 
> —
> Reply to this email directly or view it on GitHub.