-
Notifications
You must be signed in to change notification settings - Fork 47
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
Psi4 error #6
Comments
I guess this issue arises because you assume the return value of resp.resp is a two layer tuple in line 82 of helper.py. Maybe charge = resp.resp([mol], options)[1] can solve the issue? |
Hi @PigUnderRoof , yes, I discovered the same problem last week, it seems like resp has changed what the resp() function returns. I will push an update soon to enable both versions, but currently you can safely use the change you proposed ( |
Seems like I had a very old version of resp in my conda environment, all newer versions are compatible with the change - so I just went ahead and pushed that change. Thanks for reporting! |
Hi, thank you for the great work.
I meed some problem when I try to run through the exaple notebook with partialCharges='resp'. The error is about key-values of options in helper.py. I try to modify it by removing the list in line 82. Then the charge output becomes only one value rather than a list/array in line 51 of electrostatics.py. I am not an expert of Psi4, hope these description can help you understand the issue.
Appreicate it if you can help, :D!
Best,
Chao
The text was updated successfully, but these errors were encountered: