Updates:
Status: Started
Labels: NeedsReview
Comment #12 on issue 1454 by asmeurer: variable i in global namespace
http://code.google.com/p/sympy/issues/detail?id=1454
I went ahead and fixed this. It turns out that the math, random, and re
modules were being added to the global namespace
in addition to the variable "i" as a result of this. You can pull from
http://github.com/asmeurer/sympy/tree/global-i-fix. I
did not add any kind of test because I do not know what the best way to
test this would be, if it even needs to be tested at
all. Simply testing "from sympy import *" to see if it imports i and those
other things causes another test that checks for
"import *" statements to fail. We could test the massive output of
sympy.__dict__, but someone would need to go through it
first to make sure that no other garbage like this is in it. Also,
sympy.__dict__ wouldn't have caught the .k attribute that I
fixed above. Also, any time someone adds a function, they would have to
update the test.
--
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
-~----------~----~----~----~------~----~------~--~---
