-
Notifications
You must be signed in to change notification settings - Fork 2
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
Missing f-string Specifier in PhononWorkChain #61
Comments
Hello, in the main release of the code we don't have such WorkChain. I suppose you are referring to an Environ implementation, is it correct? I know a collaborator of mine that is working on this, but as you can inspect there's yet not official support. |
My mistake, I meant |
Fixes #61 The PhononWorkChain was missing an 'f' string, thus missing to report the `exit_status` of possible failed child WorkChains. The DielectricWorkChain had the `property` input badly documented.
Thanks for reporting this. I am about to open a PR to fix this. |
Fixes #61 The PhononWorkChain was missing an 'f' string, thus missing to report the `exit_status` of possible failed child WorkChains. The DielectricWorkChain had the `property` input badly documented.
In
PhononWorkChain
in theinspect_all_runs
method there is a missing f-string specifier on line 454that is causing the workchain to not report what the exit status for the subprocess that failed.
'with exit status {workchain.exit_status}'
should bef'with exit status {workchain.exit_status}'
The text was updated successfully, but these errors were encountered: