-
Notifications
You must be signed in to change notification settings - Fork 115
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
Clarify namespace for elastica modules to avoid namespace confusion and clutter #193
Comments
I think it is a good suggestion. |
Last point on pep8 |
Yes it is still open |
If this issue is still open, I would like to be assigned to it. @armantekinalp @bhosale2 |
Is this issue is open?? @armantekinalp |
@Deep-Rai-08 The issue is still open. Feel free to make a PR. |
A note to everyone interested in this issue: if you plan to work on it please either assign yourself to the issue or let us know that you wish to be assigned to the issue. |
@bhosale2 |
Hi, is anyone still working on the issue? If not I'd like to work on it. @armantekinalp |
Closed via #238. |
Currently, in example cases, the modules from
elastica
are imported directly asfrom elastica import *
. This can become an issue when usingelastica
across packages (whereelastica
is a plugin), where modules from other packages can collide in namings with the modules fromelastica
. To avoid this, it may be helpful to use functionalities inelastica
as:import elastica as ea; ea.Damping(....)
to make it explicit where the functionality comes from.@skim0119 and @armantekinalp thoughts?
The text was updated successfully, but these errors were encountered: