Comment #2 on issue 1558 by asmeurer: NumberSymbol not recognized as a filter for atoms() and I has only type Atom http://code.google.com/p/sympy/issues/detail?id=1558
As for I, I do not know if there is a good reason for I.is_NumberSymbol to be false. I would suggest trying to set it True in the ImaginaryUnit class and seeing if any tests fail. It is arguable whether this *should* be True, since I isn't really a "number", like pi or E, but it could be useful to have that for some algorithms that use is_NumberSymbol. As for atoms(), that is simple. It works by traversing through the expression and doing isinstance(expr, class), and since NumberSymbol isn't really a class, this doesn't work. You could maybe add a special case for it. By the way, should it really be is_number_symbol, since it isn't a class? -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
