Okay I made it work with __richcmp__ as suggested. Please review :-) .
On Tuesday, October 24, 2017 at 7:53:46 AM UTC+2, Simon Brandhorst wrote: > > Ahhh > > On Monday, October 23, 2017 at 6:09:02 PM UTC+2, David Roe wrote: >> >> >> >> On Mon, Oct 23, 2017 at 8:24 AM, Simon Brandhorst <sbran...@web.de> >> wrote: >> >>> 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. >> David >> > -- 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.
