Skip to content

[md] store additional properties + update tests#1976

Open
mshuaibii wants to merge 3 commits intomainfrom
md_atoms_state
Open

[md] store additional properties + update tests#1976
mshuaibii wants to merge 3 commits intomainfrom
md_atoms_state

Conversation

@mshuaibii
Copy link
Copy Markdown
Contributor

@mshuaibii mshuaibii commented Apr 22, 2026

  • Preserve atomic state in MD trajectories: Extends TrajectoryFrame to capture and serialize tags, fixed (from FixAtoms constraints), charge, and spin — fields critical for reconstructing atoms objects.
  • MD runner improvements:
    • Saves initial atoms to init_atoms.extxyz before MD begins, so the starting structure is always available.
    • Renames checkpoint files from checkpoint.xyz to checkpoint.extxyz to match the format actually being written (extxyz).
    • Adds dedicated tests for the slab+adsorbate system builder and a new TestTrajectoryFrame test class that verifies tags, constraints, charge, and spin survive a full MD → parquet round-trip.

@mshuaibii mshuaibii requested review from bkmi and lbluque April 22, 2026 18:31
@mshuaibii mshuaibii added enhancement New feature or request minor Minor version release labels Apr 22, 2026
@meta-cla meta-cla Bot added the cla signed label Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed enhancement New feature or request minor Minor version release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant