-
-
Notifications
You must be signed in to change notification settings - Fork 110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don't use import * #177
Comments
Why is it bad practice in the particular context of the sympy interactive shell to use a star import? |
Right, I always thought the |
|
I guess you could argue that maybe it isn't necessary to import all 900 of them. |
If import * is just fine in your shell then I can break it with one line. Just run this as first command from math import * # import * is just fine Then, this example from right pane no longer works: sin(x).series(x, 0, 7) # one of the examples on the right pane In general, import * is bad practice (I hope there is no disagreement here). If people see you use it, then they are likely to use it later on when writing python programs. And there it could be real bad. |
I think that it was design flaw that sympy made exceptions for |
The real design flaw is the fact that there isn't a single |
hey i can solve it please assign this issue to me |
I don't think that we want to change anything here. The online shell is designed for conveniently using sympy interactively. |
The startup code loaded by default for the SymPy shell has an import * statement. This is very very bad practice and should not be used
The text was updated successfully, but these errors were encountered: