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

IRC calculation fails under rare circumstances #18

Open
LouieSlocombe opened this issue May 4, 2023 · 2 comments
Open

IRC calculation fails under rare circumstances #18

LouieSlocombe opened this issue May 4, 2023 · 2 comments

Comments

@LouieSlocombe
Copy link

Hi there, thanks for this great package! Under some rare circumstances, the IRC calculation fails. It might be possible there is an unsafe divide?
/users/jd01548/.local/lib/python3.10/site-packages/sella/peswrapper.py:325: RuntimeWarning: invalid value encountered in divide ratio = df_actual / df_pred
I am working on a minimal working example.

@ehermes
Copy link
Collaborator

ehermes commented May 6, 2023

Is this causing an actual crash, or is it just printing that error message? I definitely should clean up some of those divisions by zero, but I don't think this should cause the code to fail entirely. A minimal working example would be greatly appreciated.

@LouieSlocombe
Copy link
Author

Thanks for the quick response and sorry for my delay! The program crashes. Both df_actual and df_pred are zero. We are using NWchem. Example

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

2 participants