Replies: 3 comments 1 reply
-
At the moment, the virials and the stress are used separately and not converted internally, which is a bad thing, and we should change that. We do divide the virial loss by the number of atoms, so the weight one provides is "per atom". |
Beta Was this translation helpful? Give feedback.
0 replies
-
so what happens when some data has stress and others have virial? I guess you need to supply loss weights for both? |
Beta Was this translation helpful? Give feedback.
1 reply
-
ok. let's create an issue and assign it to Eszter :-) |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
when we specify stress or virial, one of these presumably gets converted before being used in the loss. what happens to the loss weight? the user expectation might be that if I specify stress, then the weight is intensive, and that if I specify virial, then it's extensive. in GAP we specifically stated that the loss weights for virials are really virial/atom (same as for energy), so both are intensive in fact. what's the situation in MACE ?
Beta Was this translation helpful? Give feedback.
All reactions