Updates:
        Cc: [email protected]
        Labels: Polynomial Matrices

Comment #3 on issue 2526 by asmeurer: Making sympy __mul__s return NotImplemented wherever applicable
http://code.google.com/p/sympy/issues/detail?id=2526

Well, if it derived from Basic, that would fix the problem, because Basic defines all the __methods__.

There are some problems, though. I kind of doubt it would work correctly if you made it derive from Number. In other words, stuff wouldn't reduce modulo p all the time (like Mul(3, FF(3)(2)) vs. Mul(FF(3)(2), 3)). And what would FF(9)(3)*FF(27)(3) be (it seems this is already poorly defined; compare FF(9)(3)*FF(27)(3) with FF(27)(3)*FF(9)(3)).

If you make it derive from Expr and not Number, then it wouldn't combine into other numbers, so Mul(3, FF(3)(2)) would give 3*FF(3)(2) (right now it gives Integer(-3)).

So the question is, should it at least be Basic? I think perhaps so, but let's see what Mateusz thinks.

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sympy-issues?hl=en.

Reply via email to