OP

Added __eq__ and __hash__ to Op Class and deleted them in function with props attribute.
Yes, but I think is should be:
`super(self, Op).__hash__()`


On Tue, Aug 5, 2014 at 3:04 PM, Tanjay94 <notifications@github.com> wrote:

> In theano/gof/op.py:
>
> > @@ -575,6 +575,18 @@ def __new__(cls, *args, **kwargs):
> >      def __init__(self, use_c_code=theano.config.cxx):
> >          self._op_use_c_code = use_c_code
> >
> > +    def __hash__(self):
> > +        if hasattr(self, 'props'):
> > +            return hash((type(self), self.props()))
> > +        else:
> > +            raise utils.MethodNotDefined("c_headers", type(self), self.__class__.__name__)
>
> @abergeron <https://github.com/abergeron>
> I am sorry, but i don't know what should replace "c_headers"
> as for the hash, are you suggesting i simply write:
> else:
> super().*hash*(
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Theano/Theano/pull/2018/files#r15834192>.
>