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

Exciton Band Structure Finite Q #21

Open
alexmoratalla opened this issue Mar 2, 2022 · 2 comments
Open

Exciton Band Structure Finite Q #21

alexmoratalla opened this issue Mar 2, 2022 · 2 comments
Assignees
Labels
duplicate This issue or pull request already exists enhancement New feature or request help wanted Extra attention is needed

Comments

@alexmoratalla
Copy link
Member

I am trying to use excitondb to plot exciton band structures of finite-q BSE. However, I get the plot at the same k-point for conduction and valence band. I guess this is not implemented, so I will work in the code for that.

imagen

@alexmoratalla alexmoratalla added enhancement New feature or request help wanted Extra attention is needed labels Mar 2, 2022
@sangallidavide
Copy link
Member

sangallidavide commented Mar 2, 2022

Ciao Alejandro, I guess you are using the same implementation for q=0

In ypp when we coded the finite-q BSE we needed to get the k of the hole as well.
So at q=0 you just have the one k

     ikbz = BSS_eh_table(S_indx(neh),1)
     iv = BSS_eh_table(S_indx(neh),2)
     ic = BSS_eh_table(S_indx(neh),3)
     i_spin= spin(BSS_eh_table(S_indx(neh),:))

and one extra line to get the k in the IBZ if needed

      ikibz = Xk%sstar(ikbz,1)

At finite momentum you also need the k of the hole (first in the BZ and then eventually in the IBZ)

     ikpbz = qindx_X(iq,ikbz,1)
     ikp  = Xk%sstar(ikpbz,1)

Please notice the issue pointed out by Fulvio for correctly getting the k-points ... yambo-code/yambopy-devel#29
Also qindx_X in inside ndb.kindx. So it might be useful to read it also for this reason.

@palful
Copy link
Member

palful commented Mar 2, 2022

I think this is actually a duplicate of issue #20. I assigned that to you as well @alexmoratalla. @sangallidavide in issue #20 it is pointed out what the problem is.

Fixing a single plot manually is quite easy, but it would be nice to implement matrix element rotation in yambopy. Issue yambo-code/yambopy-devel#29 may be relevant because of the indexing inconsistency as you point out. The main problem is that the code that interpolates the weights expects always a k-point in the IBZ because it was done with only the q=0 in mind.

@palful palful added the duplicate This issue or pull request already exists label Mar 2, 2022
@palful palful transferred this issue from another repository Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants