Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix real-space Wannier function normalization #2

Open
2 of 4 tasks
eassmann opened this issue Dec 22, 2015 · 2 comments
Open
2 of 4 tasks

fix real-space Wannier function normalization #2

eassmann opened this issue Dec 22, 2015 · 2 comments

Comments

@eassmann
Copy link
Member

eassmann commented Dec 22, 2015

In wplot, the Wannier functions are currently not normalized. Rather, ⟨w|w⟩ = Nk X where Nk is the number of k-points and X an unknown factor.

Additionally, in tests with SOC, there is a factor (~2.16) between a setup with and one without RLOs.

  • Take number of k-points into account.
  • Figure out what X is and take it into account as well.
  • Figure out RLO-induced factor.
  • Document in user's guide.
@eassmann eassmann self-assigned this Dec 22, 2015
@eassmann eassmann added this to the 1.1.0 milestone Dec 22, 2015
@eassmann eassmann added the wplot label Dec 30, 2015
@eassmann eassmann changed the title Fix real-space Wannier function normalization fix real-space Wannier function normalization Dec 30, 2015
@eassmann
Copy link
Member Author

eassmann commented Feb 3, 2016

Normalization by #k now fixed (f14e29c).

eassmann referenced this issue Dec 3, 2016
Previously, `w(r)` as printed in `case.psink` was normalized as
`|w²(r)|/#k` leading to a factor `#k`.  Fix this to `|w(r)/#k|²`.

Thanks to Ulrich Wedig.
@eassmann
Copy link
Member Author

The remaining factor is simply due to the units. If AU units are requested in case.inwplot, then sum(case.psink) * dV ≈ 1 holds, where in the simplest case dV = (L / [Np - 1])³ with axis length L as given in the header of case.psink and Np points in each direction.

If ANG units are selected, L must also be in Å, of course. Otherwise, an extra factor of (Å/Bohr)³ ≈ 6.75 appears.

The RLO-related factor remains to be tested in 2.0.

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

No branches or pull requests

1 participant