Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Sign issue in stress calculations #10

Open
amirhajibabaei opened this issue Nov 14, 2019 · 0 comments
Open

Sign issue in stress calculations #10

amirhajibabaei opened this issue Nov 14, 2019 · 0 comments
Labels
Theoretical Something works but it may be a wrong

Comments

@amirhajibabaei
Copy link
Owner

Theoretically
Stress=stress1 + stress2,
where
stress1 = (sum over all atoms F.T@r) / volume
where r is coordinates and F forces on atom.
stress2 is related to derivatives wrt cell.
This works fine in ParametricCalculator (test using calc.calculate_numerical_stress(atoms)).
But in AutoForceCalculator (machine learned) we have to multiply stress1 by -1 in order to get the right numerical stresses.
Everything seemingly works just fine but I still can't explain the -1 multiplyer.

@amirhajibabaei amirhajibabaei added the Theoretical Something works but it may be a wrong label Nov 14, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Theoretical Something works but it may be a wrong
Projects
None yet
Development

No branches or pull requests

1 participant