Skip to content
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

GMSO Broken issue? #540

Closed
CalCraven opened this issue May 3, 2021 · 1 comment
Closed

GMSO Broken issue? #540

CalCraven opened this issue May 3, 2021 · 1 comment

Comments

@CalCraven
Copy link
Contributor

I merged in some of the most recent commits to try to finalize PR #492 and realized that I couldn't properly import gmso to check functionality. I cloned a fresh repo with the most recent commit 893534b, and see the same import error.

Code to reproduce error:

import gmso

Traceback (most recent call last):
File "", line 1, in
File "/Users/calcraven/Documents/Vanderbilt/Research/git_repos/gmso_check_import_error/gmso/gmso/init.py", line 1, in
from .core.topology import Topology
File "/Users/calcraven/Documents/Vanderbilt/Research/git_repos/gmso_check_import_error/gmso/gmso/core/topology.py", line 7, in
from gmso.core.atom import Atom
File "/Users/calcraven/Documents/Vanderbilt/Research/git_repos/gmso_check_import_error/gmso/gmso/core/atom.py", line 8, in
from gmso.core.element import Element
File "/Users/calcraven/Documents/Vanderbilt/Research/git_repos/gmso_check_import_error/gmso/gmso/core/element.py", line 301, in
elements_dict = {int(key): value for key, value in elements_dict.items()}
File "/Users/calcraven/Documents/Vanderbilt/Research/git_repos/gmso_check_import_error/gmso/gmso/core/element.py", line 301, in
elements_dict = {int(key): value for key, value in elements_dict.items()}


It looks like the most recent changes to gmso that effect element.py come from #536. I might be getting something really wrong, but if there is an issue this is something that should be addressed asap.


Version information:
python = 3.7
mbuild = 0.10.13
foyer = 0.7.6
gmso = 0.5.0.dev0

@CalCraven
Copy link
Contributor Author

Seems like the change is in ele.
Version used :
ele = 0.1.0
Version needed:
ele = 0.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant