Skip to content

I have changed the sort_by_id to be True by default, instead of False, in ase/io/lammpsdata.py

M K requested to merge MKsutov/ase:MKsutov-master-patch-13543 into master

Please merge this. If one wishes to save time by toggling the option off, they should do it themselves. Initially, I imagine most people won't know of this option's existence. It is dangerous to leave sort_by_id=False, as this can lead to atomic elements swapping places (very dangerous when elemental ordering is crucial, like in short-range ordered alloys)

Merge request reports