Skip to content

Commit

Permalink
Gross mistake in Gamma-only stress for hybrid functionals
Browse files Browse the repository at this point in the history
  • Loading branch information
giannozz committed Jun 23, 2018
1 parent ae63562 commit 5ed1486
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
4 changes: 3 additions & 1 deletion Doc/release-notes
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ New in development version:

* Spin-polarized X3LYP (experimental)

* EPW: Electronic transport (Botlzmann transport equation)
* EPW: Electronic transport (Boltzmann transport equation)
Iterative Botlzmann transport equation (experimental)
Cumulant treatement (model)
Possible screening
Expand All @@ -21,6 +21,8 @@ New in development version:

Problems fixed in development version:

* Stress with hybrid functionals and Gamma point was grossly wrong

* EPW: Velocity in the dipole approximation was unstable because of
G-vector ordering.

Expand Down
4 changes: 2 additions & 2 deletions PW/src/exx.f90
Original file line number Diff line number Diff line change
Expand Up @@ -2634,11 +2634,11 @@ FUNCTION exx_stress()
!$omp parallel do default(shared), private(ig), reduction(+:vc)
DO ig = 1, ngm
!
vc(:,:) = vc(:,:) + fac(ig) * x1 * &
vc(:,:) = vc(:,:) + x1 * 0.25_dp * &
abs( rhoc(dfftt%nl(ig)) + &
conjg(rhoc(dfftt%nlm(ig))))**2 * &
(fac_tens(:,:,ig)*fac_stress(ig)/2.d0 - delta(:,:)*fac(ig))
vc(:,:) = vc(:,:) + fac(ig) * x2 * &
vc(:,:) = vc(:,:) + x2 * 0.25_dp * &
abs( rhoc(dfftt%nl(ig)) - &
conjg(rhoc(dfftt%nlm(ig))))**2 * &
(fac_tens(:,:,ig)*fac_stress(ig)/2.d0 - delta(:,:)*fac(ig))
Expand Down

0 comments on commit 5ed1486

Please sign in to comment.