-
Notifications
You must be signed in to change notification settings - Fork 1
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
Check if the calculation finished properly #17
Comments
This means the method.smearing_width is currently unitless, this is IMO a bug and should be fixed upstream. I've reported this in nomad-coe/nomad#19 however for now just don't use .magnitude. |
what about the test case that you mentioned u will add some. |
Yeah, I'm now not sure how it actually works, it seems the .out file is produced only at the end of the calculations (at least for small calculations), so I can't find a partial out file right now. I have some old cases which seem to be incomplete, but they are several hundreds of MB big, so not something we would want to have in git. Just test the run_clean_end = True scenario for now. |
Fixed with c67f954 |
The .out file should end with timing section. If we have it, it means the calculation finished properly. Checking for the last line is probably enough.
If we see this complete section we should set
section_run.run_clean_end = True.
and False otherwise. I'll add some testcase which exercises the False incomplete out file scenario.The text was updated successfully, but these errors were encountered: