>
> What exactly was the tuple before?
>
The tuple before was (auto_symbol, auto_number, factorial_notation). The
new function that replaces the tuple iterates through those functions
internally and applies each of them.
> Did you also fix comment 3 of 2663?
>
I added those cases as tests, actually.
> Can you make your fix generalized to an API? I need this exact same fix
> to change isympy -a to use parsing.
>
>
Can you clarify? So you want the fix refactored by itself? I'm not sure if
that warrants a separate function, the change is simply (while iterating
through the tokens)
if (# Don't convert attribute access
(prevTok[0] == OP and prevTok[1] == '.') or
# Don't convert keyword arguments
(prevTok[0] == OP and prevTok[1] in ('(', ',')
and nextTokNum == OP and nextTokVal == '=')):
# Don't insert call to Symbol
else:
# Insert call to Symbol
I'll put up a pull request soon.
David
--
You received this message because you are subscribed to the Google Groups
"sympy" group.
To view this discussion on the web visit
https://groups.google.com/d/msg/sympy/-/HRg1zva3LesJ.
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?hl=en.
