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

bugfix: UNT calculation #2

Open
kzm7k6r opened this issue Jan 12, 2019 · 3 comments
Open

bugfix: UNT calculation #2

kzm7k6r opened this issue Jan 12, 2019 · 3 comments

Comments

@kzm7k6r
Copy link
Collaborator

kzm7k6r commented Jan 12, 2019

The ic loop should be inside. Pls confirm
!
!20181211 Kazuma Nakamura
!
!do ic=1,ncomp
do ik=1,NTK
do ib=1,N_BAND(ik)
do jb=1,n_occ
!ic=1
SUM_CMPX=0.0d0
do ic=1,ncomp !20181211 Kazuma Nakamura
do ig=1,NG0(ik)
SUM_CMPX
+ =SUM_CMPX
+ +conjg(C0(ig,ic,ib+N_BAND_BTM(ik),ik))*C0WN(ig,ic,jb,ik)
! + +conjg(C0(ig,ic,ib+N_BAND_BTM(ik),ik))*C0WN(ig,jb,ik)
enddo!ig
enddo!ic
UNT(ib,jb,ik)=SUM_CMPX
enddo!jb
enddo!ib
enddo!ik
!enddo!ic

@mnmpdadish
Copy link
Owner

I think that you are right about this bugfix. I included it in the version, but I need/want to test it to see if this is correct. This should have a significant effect on the result of chiqw. TBC.

@kzm7k6r
Copy link
Collaborator Author

kzm7k6r commented Feb 6, 2019

When doing cRPA, we have to fix this bug.

@mnmpdadish
Copy link
Owner

ok I test a cRPA case then, and check the difference.

mnmpdadish pushed a commit that referenced this issue Jun 7, 2019
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

No branches or pull requests

2 participants