Change the config_from_atoms function such that you can also run .traj files and other trajectory file types about mace HOT 7 CLOSED

Hi! Sorry, I am not sure I understand what you changed?
could you perhaps explain a little more?

from mace.

Actually I think there might be a mistake in the function I wrote.
It works fine for training, but in the MD prediction it gives an error.

The hole idea of my implementation was to be able to use ASE trajectory files :) Do you think that can be made possible?

from mace.

Could you post the error you get when trying to load your traj file? If ase.io.read works the same way on them, I am not sure where the problem might be.

from mace.

The energy and forces will not be loaded, since they do not have the atoms.info and atoms.arrays dictionary :) The will not be an error by it self but the model is not training.

from mace.

The problem with the command extraction is that it can be unreliable. If a calculator is attached to the atom object, it can give inconsistent results. For this reason, storing your data in the info with a custom key is highly recommended. In particular, .traj files are usually associated with the calculator that generated them. Is there an easy way to convert the .traj to .xyz? We could add this pre-processing.

from mace.

I think traj files store those quantities in a SinglePointCalculator, which is an essentially static storage that fills in the return values for get_potential_energy() etc. In fact, it's probably safer than quantities from extxyz, because I think the calculator is invalidated if you modify positions etc. It's just not possible to do things like save multiple sets of results from different calculators. It would, of course, be trivial to write a little loop that reads the calculator and stores the results in info/arrays. The ase.io extxyz reader probably already does that, I think.

from mace.

I think converting traj to xyz file will require mor space since you will have the same data file twice.
What Bernstei suggest would be a great solution!

from mace.