Skip to content

DeePKS should update pdm before outputting labels #5856

@xuan112358

Description

@xuan112358

Describe the bug

When outputting DeePKS labels, pdm isn't updated using dm after diagonalization. This results in gevdm and precalc of force/stress/bandgap/hamiltonian being calculated using the older dm.
However, Hamiltonian matrix and other things are calculated using the new dm.
This may cause inconsistency.

Expected behavior

DeePKS should update pdm before outputting labels

To Reproduce

No response

Environment

No response

Additional Context

No response

Task list for Issue attackers (only for developers)

  • Verify the issue is not a duplicate.
  • Describe the bug.
  • Steps to reproduce.
  • Expected behavior.
  • Error message.
  • Environment details.
  • Additional context.
  • Assign a priority level (low, medium, high, urgent).
  • Assign the issue to a team member.
  • Label the issue with relevant tags.
  • Identify possible related issues.
  • Create a unit test or automated test to reproduce the bug (if applicable).
  • Fix the bug.
  • Test the fix.
  • Update documentation (if necessary).
  • Close the issue and inform the reporter (if applicable).

Metadata

Metadata

Assignees

Labels

Machine LearningIssues related to the DeePKSRefactorRefactor ABACUS codes

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions