> I am aware of this. It is just unexpected that other.__eq__(self) calls >> self._eq(other) >> >> As a user of that rich cmp method I would have expected it to call >> other._eq(self). >> >> Raising a not implemented error seems nice. :) >> > > Note that it should return NotImplemented (a built in constant to be > returned by rich comparisons), not raise a NotImplementedError. > > There is also precedent in Python for raising a TypeError with < and > too. However, I think that causes more headaches than it solves.
Best, Travis -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at https://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/d/optout.