commonsense team mailing list archive

[Ken Arnold <kenneth.arnold@xxxxxxxxx>]

Public bug reported:

Our DictMixin defines a __cmp__. It shouldn't, because tensor1 > tensor2
doesn't make sense. __eq__ and __ne__ should be the only comparison
operators implemented, and there are much more efficient ways to
accomplish them than the approach used in DictMixin.

** Affects: divisi
     Importance: Undecided
         Status: New

-- 
Tensors shouldn't be comparable
https://bugs.launchpad.net/bugs/471416
You received this bug notification because you are a member of
Commonsense Computing, which is the registrant for Divisi.

Status in Divisi: New

Bug description:
Our DictMixin defines a __cmp__. It shouldn't, because tensor1 > tensor2 doesn't make sense. __eq__ and __ne__ should be the only comparison operators implemented, and there are much more efficient ways to accomplish them than the approach used in DictMixin.