choderalab / modelforge

Infrastructure to implement and train NNPs
https://modelforge.readthedocs.io/en/latest/
MIT License
9 stars 4 forks source link

fix indexing bug #87

Closed wiederm closed 3 months ago

wiederm commented 3 months ago

Description

Torchani used atom index instead of atomic numbers to aggregate the element specific atomic contributions. Sidenote: we should standardize this between the NNPs at some point, and always use atomic numbers for index operations.