Skip to content

Feature: Implement moving spatial gauge for RT-TDDFT Ehrenfest dynamics (developer preview)#7300

Merged
mohanchen merged 1 commit intodeepmodeling:developfrom
AsTonyshment:ehrenfest
Apr 30, 2026
Merged

Feature: Implement moving spatial gauge for RT-TDDFT Ehrenfest dynamics (developer preview)#7300
mohanchen merged 1 commit intodeepmodeling:developfrom
AsTonyshment:ehrenfest

Conversation

@AsTonyshment
Copy link
Copy Markdown
Collaborator

@AsTonyshment AsTonyshment commented Apr 29, 2026

Description

This PR introduces the underlying implementation of the Moving Spatial Gauge for RT-TDDFT Ehrenfest dynamics.

When using atom-centered Numerical Atomic Orbitals (NAOs), the basis functions move with the atomic coordinates, introducing nonadiabatic coupling terms (the $P$ matrix). This implementation compensates for these terms to ensure strict Total Energy Conservation (EC) during highly nonadiabatic time-dependent evolutions.

Implementation Details

  • Added a hardcoded switch bool use_td_moving_gauge = false; in source/source_esolver/esolver_ks_lcao_tddft.h.
  • This feature is currently not available to standard users. By default, the code bypasses the construction and application of the moving spatial gauge matrix. It is strictly reserved for developer debugging and rigorous testing.

Validation & Test Results

I performed a 200 fs time-evolution test on a vibrating NaCl diatomic molecule. By enabling the moving spatial gauge, the unphysical energy drift is effectively eliminated. The total energy fluctuation drops dramatically from ~579 meV to ~2.1 meV, achieving a conservation precision closely approaching the BOMD reference.
md_evolution

Copilot AI review requested due to automatic review settings April 29, 2026 15:07
@AsTonyshment AsTonyshment requested review from ESROAMER and mohanchen and removed request for Copilot April 29, 2026 15:08
Copy link
Copy Markdown
Collaborator

@mohanchen mohanchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just lack some tests

@mohanchen mohanchen added Features Needed The features are indeed needed, and developers should have sophisticated knowledge Refactor Refactor ABACUS codes labels Apr 30, 2026
@mohanchen mohanchen merged commit a8413b3 into deepmodeling:develop Apr 30, 2026
19 checks passed
@AsTonyshment AsTonyshment deleted the ehrenfest branch April 30, 2026 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Features Needed The features are indeed needed, and developers should have sophisticated knowledge Refactor Refactor ABACUS codes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants