Skip to content

Conversation

@WHUweiqingzhou
Copy link
Collaborator

Fix #5011

The List of Change

  1. Before this PR, SCF converges for drho < scf_thr || dene < scf_ene_thr. After this PR, SCF converges for drho < scf_thr && dene < scf_ene_thr.
  2. Default value scf_ene_thr = 1.0

@QuantumMisaka
Copy link
Collaborator

@WHUweiqingzhou Can we allow users to choose the AND and OR method linking two thresholds?

@WHUweiqingzhou
Copy link
Collaborator Author

@WHUweiqingzhou Can we allow users to choose the AND and OR method linking two thresholds?

In the cases where you prefer energy convergence, you can set a normal scf_ene_thr=1e-7, and set a large scf_thr=1e-4.

@WHUweiqingzhou
Copy link
Collaborator Author

Close this one, and see #5357

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Energy is converge to 1e-7, but drho only to 1e-3

3 participants