aiida-quantumespresso icon indicating copy to clipboard operation
aiida-quantumespresso copied to clipboard

🐛 BFGS double reset failure not parsed properly

Open mbercx opened this issue 8 months ago • 1 comments

In a not-so-recent commit, the error message related to a double reset of the history of the BFGS algorithm was changed in some cases in the output of Quantum ESPRESSO to print exiting instead of stopping:

https://gitlab.com/QEF/q-e/-/commit/cd6e88f56e2325c944a284e9738b23908d122764

This means we no longer catch this issue, since we look for the following string:

https://github.com/aiidateam/aiida-quantumespresso/blob/210c40bbc3445f55155bbb855d320afa00fa347e/src/aiida_quantumespresso/parsers/parse_raw/pw.py#L246

So these failures have been silently passing as successful calculations.

Reported by @Minotakm.

mbercx avatar May 29 '24 14:05 mbercx