Skip to content

Fix the issue of inconsistent behavior between estep and md_nstep.#7198

Merged
mohanchen merged 10 commits intodeepmodeling:developfrom
ESROAMER:develop
Apr 2, 2026
Merged

Fix the issue of inconsistent behavior between estep and md_nstep.#7198
mohanchen merged 10 commits intodeepmodeling:developfrom
ESROAMER:develop

Conversation

@ESROAMER
Copy link
Copy Markdown
Collaborator

@ESROAMER ESROAMER commented Apr 1, 2026

Linked Issue

Fix #7068

What's changed?

  1. fix RT-TDDFT calculations with hybrid functional PBE0 were very different when setting md_nsteps and estep_per_md #7068. Now the two results are consistent.
image
  1. Add an integration test for td+exx calculations.
  2. Fixed a bug where cal_DMR was called incorrectly, which could affect the results of long-term evolution under the hybrid gauge.

@mohanchen mohanchen requested a review from AsTonyshment April 2, 2026 01:15
@mohanchen mohanchen added Bugs Bugs that only solvable with sufficient knowledge of DFT Refactor Refactor ABACUS codes labels Apr 2, 2026
Copy link
Copy Markdown
Collaborator

@AsTonyshment AsTonyshment left a comment

Choose a reason for hiding this comment

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

Nice work. LGTM.

@mohanchen mohanchen merged commit bf92c2d into deepmodeling:develop Apr 2, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugs Bugs that only solvable with sufficient knowledge of DFT Refactor Refactor ABACUS codes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RT-TDDFT calculations with hybrid functional PBE0 were very different when setting md_nsteps and estep_per_md

3 participants