stfc / janus-core

Tools for machine learnt interatomic potentials
https://stfc.github.io/janus-core/
BSD 3-Clause "New" or "Revised" License
9 stars 7 forks source link

Add SinglePoint.run setting for invalidate_calc? #184

Closed ElliottKasoar closed 1 month ago

ElliottKasoar commented 2 months ago

Currently this invalidate_calc isn't settable, so there's no way to keep calculated results in struct.calc.results.

It might be nice to add it as an option.

If we do, I think we will need to set pass write_results=False to ase.io.write, otherwise the same problem seen in #182/#183 will probably reappear. (I think it probably makes sense to do this regardless, since we manually add all the results to info/arrays, so this removes the possibility of duplication for further variables.)