Hello!
Thanks for bringing this up again because the information previously communicated in the topic you referenced seems to be incorrect. I have to disagree with the statements of my colleague: VASP does and always did check the convergence of the ab initio calculation. Only if it is converged the energies, forces and stress are added to the reference data (in the ML_ABN file) which is used for refining the force field. This safety measure has been implemented since the first release of ML force fields in VASP 6.3.0. On the source code level this can be seen when following the LABORT variable being passed down into subroutines of ml_ff_abinitio.F. The comments and the if-conditions we can find there clearly indicate that the reference data collection is "protected" against picking up non-converged data. I quickly tested this with a simple system and setting NELM to a low value, triggering multiple non-converged steps within a short trajectory with this screen message:
Code: Select all
-----------------------------------------------------------------------------
| |
| W W AA RRRRR N N II N N GGGG !!! |
| W W A A R R NN N II NN N G G !!! |
| W W A A R R N N N II N N N G !!! |
| W WW W AAAAAA RRRRR N N N II N N N G GGG ! |
| WW WW A A R R N NN II N NN G G |
| W W A A R R N N II N N GGGG !!! |
| |
| The electronic self-consistency was not achieved in the given |
| number of steps (NELM). The forces and other quantities evaluated |
| might not be reliable so examine the results carefully. If you find |
| spurious results, we suggest increasing NELM, if you were close to |
| convergence or switching to a different ALGO or adjusting the |
| density mixing parameters otherwise. |
| |
-----------------------------------------------------------------------------
As expected, in the affected ionic steps the "faulty" data is not used for training and does not make it into the ML_ABN file.
What remains to be added in an upcoming VASP release is an easy-to-parse message in the ML_LOGFILE, signalling each non-converged ionic step. We have this on our ToDo list and I just updated the Wiki about the current behavior.
I am sorry for the inconvenience and the incorrect information you received previously about this issue. I hope it is at least reassuring that this "feature" has been implemented from the start.
All the best,
Andreas Singraber