2010/3/29 Øyvind Jensen <jensen.oyv...@gmail.com>: >> I think he meant to put commutative=False, not noncommutative=True. > > Exactly, I put up another branch fix__lt__2, where this is fixed.
Thanks! I'm +1 for this one. I have only a few minor stylistic remarks: + assert (a<b) == Lt(a,b) + assert (a<=b) == Le(a,b) + assert (a>b) == Gt(a,b) + assert (a>=b) == Ge(a,b) + if cmp(a,b) > 0: The parentheses after the assert are syntactically not necessary. Before and after any operator or after a comma should be usually a space. Vinzent -- You received this message because you are subscribed to the Google Groups "sympy-patches" group. To post to this group, send email to sympy-patc...@googlegroups.com. To unsubscribe from this group, send email to sympy-patches+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/sympy-patches?hl=en.