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

Psi4 error reporting largely unusable in production workloads #279

Open
dotsdl opened this issue Nov 5, 2020 · 0 comments
Open

Psi4 error reporting largely unusable in production workloads #279

dotsdl opened this issue Nov 5, 2020 · 0 comments

Comments

@dotsdl
Copy link
Collaborator

dotsdl commented Nov 5, 2020

Describe the bug

We previously attempted to improve the error reporting of the Psi4Harness via #266. However, now most of the errors we observe in production use of the Psi4Harness called within geomeTRIC are of the catchall variety, namely: "Unknown error, error message is not found"

Is it possible that the keys we are attempting to access in Psi4Harness._handle_errors are simply not given by the output structures of the psiapi? Are we trying to access the error messages incorrectly?

To Reproduce

This is difficult to reproduce, as the errors are likely legitimate and occur in our use case from problematic geometry optimizations. Someone perhaps more familiar with the structures of the psiapi in psi4 may be well-positioned to validate the usage of the output structure to extract error messages in Psi4Harness._handle_errors. This could be as simple as a typo in a dictionary key name.

Expected behavior

We expect more detailed error messages to be found in many cases, but a large fraction of our Psi4 errors via geomeTRIC are of the "Unknown error" variety. This is unexpected and largely not useful, since this catchall error message is meant to be the exception, not the rule.

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