-
Notifications
You must be signed in to change notification settings - Fork 15
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
Explicitly specify water potential #353
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Pull Request Test Coverage Report for Build 1213865548
💛 - Coveralls |
It turns out that the integration tests fail because the TIP3P water potential, which is now explicitly specified in the notebook, is not available: ~/work/pyiron_atomistics/pyiron_atomistics/pyiron_atomistics/atomistics/job/potentials.py in find_by_name(self, potential_name)
74 mask = (self._potential_df["Name"] == potential_name)
75 if not mask.any():
---> 76 raise ValueError("Potential '{}' not found in database.".format(
77 potential_name))
78 return self._potential_df[mask]
ValueError: Potential 'H2O_tip3p' not found in database. I think the reason is that these potentials are not included in pyiron-resources. @jan-janssen, this also explains #349. When the water potentials are not found, pyiron tries to search for alternate potentials with |
@sudarsan-surendralal Can not you define the water potential, just like you did in https://github.com/pyiron/pyiron_atomistics/blob/master/notebooks/water_MD.ipynb ? |
Thanks this should work now! |
Could possible fix the failing integration notebooks!