You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the class EnumerateStructureTransformation (defined in pymatgen/transformations/advanced_transformations.py), the M3GNetCalculator is called like this:
m3gnet_model = M3GNetCalculator(potential=potential, stress_weight=0.01).
However, the stress_weight value should be either 1.0 (for GPa) or 1 / 160.21766208 (for eV/A^3).
Expected Behavior
Either:
m3gnet_model = M3GNetCalculator(potential=potential). (--> default value of 1.0 would be used),
or:
m3gnet_model = M3GNetCalculator(potential=potential, stress_weight=1/160.21766208).
Python version
3.12.4
Pymatgen version
2024.10.3
Operating system version
No response
Current behavior
In the class EnumerateStructureTransformation (defined in pymatgen/transformations/advanced_transformations.py), the M3GNetCalculator is called like this:
m3gnet_model = M3GNetCalculator(potential=potential, stress_weight=0.01).
However, the stress_weight value should be either 1.0 (for GPa) or 1 / 160.21766208 (for eV/A^3).
Expected Behavior
Either:
m3gnet_model = M3GNetCalculator(potential=potential). (--> default value of 1.0 would be used),
or:
m3gnet_model = M3GNetCalculator(potential=potential, stress_weight=1/160.21766208).
Minimal example
Relevant files to reproduce this bug
Since it is not a bug but an eventual little mistake, no file is provided.
The text was updated successfully, but these errors were encountered: