Create a debug/md branch based on current enhance, and try to fix the bugs in current MD module, upload the visualized results to this channel after implementation and tests. Fix them one by one not in one shot. @AxiEj
-
High: The velocity-rescale thermostat formula is still suspect. The current enhance/dev branch retains this risk in vrescale.py (line 113). This was intentionally left untouched in the current merge round, as the conclusions for MD canonical sampling are not yet stable enough for release.
-
High: NPT still degrades to kinetic-only pressure when stress information is unavailable (utils.py, line 133). For common in-tree calculators such as MACE and AIMNet, this means "it runs" does not imply "physically acceptable."
-
Medium: Degrees-of-freedom bookkeeping is still hardcoded as 3N / 3N−3 logic and does not fully align with remove_com / remove_rotation settings (utils.py, line 68; vrescale.py, line 79).
-
Medium: On a fresh run the logger still backs up the current main .out file, which can split a single run's log across two files (logger.py, line 180).
Create a debug/md branch based on current enhance, and try to fix the bugs in current MD module, upload the visualized results to this channel after implementation and tests. Fix them one by one not in one shot. @AxiEj
High: The velocity-rescale thermostat formula is still suspect. The current
enhance/devbranch retains this risk invrescale.py(line 113). This was intentionally left untouched in the current merge round, as the conclusions for MD canonical sampling are not yet stable enough for release.High: NPT still degrades to kinetic-only pressure when stress information is unavailable (
utils.py, line 133). For common in-tree calculators such as MACE and AIMNet, this means "it runs" does not imply "physically acceptable."Medium: Degrees-of-freedom bookkeeping is still hardcoded as 3N / 3N−3 logic and does not fully align with
remove_com/remove_rotationsettings (utils.py, line 68;vrescale.py, line 79).Medium: On a fresh run the logger still backs up the current main
.outfile, which can split a single run's log across two files (logger.py, line 180).