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.

Reply via email to