-
Notifications
You must be signed in to change notification settings - Fork 5
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
Can the ObjectContainer class be removed? #56
Comments
I investigated TraP wrt the ObjectContainer class. It can be removed there and will be after this PR is merged. |
An easy solution may be to add the Downside is that you have to add the same method twice. |
Why is this closed? |
Not sure, I think it got closed automatically, or maybe I mistook it for another issue. |
@suvayu noticed that the ObjectContainer class has little more than a
closest_to
method which raises the question if this class could be removed.I will try to find out what breaks in PySE and in TraP if we remove it.
The text was updated successfully, but these errors were encountered: