Merge lp:~nickpapior/siesta/4.0-xc-np into lp:~albertog/siesta/4.0-xc

Proposed by Nick Papior
Status: Merged
Approved by: Alberto Garcia
Approved revision: 526
Merged at revision: 526
Proposed branch: lp:~nickpapior/siesta/4.0-xc-np
Merge into: lp:~albertog/siesta/4.0-xc
Diff against target: 97 lines (+13/-18)
3 files modified
Src/SiestaXC/ggaxc.f (+2/-5)
Src/xc.f (+10/-12)
version.info (+1/-1)
To merge this branch: bzr merge lp:~nickpapior/siesta/4.0-xc-np
Reviewer Review Type Date Requested Status
Alberto Garcia Approve
Review via email: mp+326621@code.launchpad.net

Description of the change

Removed unnecessary comments.

No logic is changed in the code.

To post a comment you must log in.
Revision history for this message
Alberto Garcia (albertog) wrote :

OK. Approved.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Src/SiestaXC/ggaxc.f'
--- Src/SiestaXC/ggaxc.f 2017-06-30 14:45:53 +0000
+++ Src/SiestaXC/ggaxc.f 2017-06-30 18:01:40 +0000
@@ -130,11 +130,9 @@
130 if (nSpin==4) then130 if (nSpin==4) then
131 NS = 2 ! Diagonal spin components131 NS = 2 ! Diagonal spin components
132132
133CC
134 if ( old_scheme ) then133 if ( old_scheme ) then
135 NS = 2
136 DTOT = D(1) + D(2)134 DTOT = D(1) + D(2)
137 dpolz= D(1)-D(2)135 dpolz= D(1) - D(2)
138 dpolxy= 2.0d0*sqrt(d(3)**2+d(4)**2)136 dpolxy= 2.0d0*sqrt(d(3)**2+d(4)**2)
139 dpol = sqrt( dpolz**2 + dpolxy**2 )137 dpol = sqrt( dpolz**2 + dpolxy**2 )
140 if ( dpol.gt.1.0d-12 ) then138 if ( dpol.gt.1.0d-12 ) then
@@ -158,7 +156,7 @@
158 GDD(IX,2) = GD(IX,1)*S2**2 + GD(IX,2)*C2**2 -156 GDD(IX,2) = GD(IX,1)*S2**2 + GD(IX,2)*C2**2 -
159 . 2.d0*C2*S2*(GD(IX,3)*CP - GD(IX,4)*SP)157 . 2.d0*C2*S2*(GD(IX,3)*CP - GD(IX,4)*SP)
160 ENDDO158 ENDDO
161CC159
162 else160 else
163161
164 ! Find eigenvalues of density matrix Dij (diagonal densities DD, i.e.162 ! Find eigenvalues of density matrix Dij (diagonal densities DD, i.e.
@@ -215,7 +213,6 @@
215 . + dGDPOLdGD(:) ) / 2213 . + dGDPOLdGD(:) ) / 2
216 dGDDdGD(2,:) = ( dGDTOTdGD(:) ! dGradDensDn/dGradD(i)214 dGDDdGD(2,:) = ( dGDTOTdGD(:) ! dGradDensDn/dGradD(i)
217 . - dGDPOLdGD(:) ) / 2215 . - dGDPOLdGD(:) ) / 2
218CC
219 endif216 endif
220 217
221 else if (nSpin==1 .or. nSpin==2) then ! Normal (collinear) spin218 else if (nSpin==1 .or. nSpin==2) then ! Normal (collinear) spin
222219
=== modified file 'Src/xc.f'
--- Src/xc.f 2017-06-30 14:37:45 +0000
+++ Src/xc.f 2017-06-30 18:01:40 +0000
@@ -406,7 +406,6 @@
406 . VPOL, DECDGN(3,2), DEXDGN(3,2),406 . VPOL, DECDGN(3,2), DEXDGN(3,2),
407 . C2, S2, ST, CT, CP, SP, dpolz, dpolxy407 . C2, S2, ST, CT, CP, SP, dpolz, dpolxy
408408
409CC RC Added: CT, dpolz, dpolxy
410409
411 PARAMETER ( TINY = 1.D-12 )410 PARAMETER ( TINY = 1.D-12 )
412411
@@ -428,12 +427,11 @@
428 else427 else
429 THETA = 0.0_dp428 THETA = 0.0_dp
430 endif429 endif
431CC RC----------------
432 C2 = COS(THETA/2)430 C2 = COS(THETA/2)
433 S2 = SIN(THETA/2)431 S2 = SIN(THETA/2)
434 ST = SIN(THETA)432 ST = SIN(THETA)
435 CT = COS(THETA) ! CC RC Added433 CT = COS(THETA)
436 PHI = ATAN2(-D(4),D(3)) ! CC RC Modified 434 PHI = ATAN2(-D(4),D(3))
437 CP = COS(PHI)435 CP = COS(PHI)
438 SP = SIN(PHI)436 SP = SIN(PHI)
439437
@@ -494,21 +492,21 @@
494 IF (nspin .EQ. 4) THEN492 IF (nspin .EQ. 4) THEN
495C Find dE/dD(ispin) = dE/dDup * dDup/dD(ispin) +493C Find dE/dD(ispin) = dE/dDup * dDup/dD(ispin) +
496C dE/dDdown * dDown/dD(ispin)494C dE/dDdown * dDown/dD(ispin)
497CC RC Note convention: 495C Note convention:
498C DEDD(1)=dE/dD11, DEDD(2)=dE/dD22,496C DEDD(1)=dE/dD11, DEDD(2)=dE/dD22,
499C DEDD(3)=Re(dE/dD12)=Re(dE/dD21), 497C DEDD(3)=Re(dE/dD12)=Re(dE/dD21),
500C DEDD(4)=Im(dE/dD12)=-Im(dE/D21)498C DEDD(4)=Im(dE/dD12)=-Im(dE/D21)
501CC RC499C
502 VPOL = (DEXDN(1)-DEXDN(2)) * CT ! RC Modified500 VPOL = (DEXDN(1)-DEXDN(2)) * CT
503 DEXDD(1) = 0.5D0 * ( DEXDN(1) + DEXDN(2) + VPOL )501 DEXDD(1) = 0.5D0 * ( DEXDN(1) + DEXDN(2) + VPOL )
504 DEXDD(2) = 0.5D0 * ( DEXDN(1) + DEXDN(2) - VPOL )502 DEXDD(2) = 0.5D0 * ( DEXDN(1) + DEXDN(2) - VPOL )
505 DEXDD(3) = 0.5d0 * (DEXDN(1)-DEXDN(2)) * ST * CP ! RC Modified503 DEXDD(3) = 0.5d0 * (DEXDN(1)-DEXDN(2)) * ST * CP
506 DEXDD(4) =-0.5d0 * (DEXDN(1)-DEXDN(2)) * ST * SP ! RC Modified504 DEXDD(4) =-0.5d0 * (DEXDN(1)-DEXDN(2)) * ST * SP
507 VPOL = (DECDN(1)-DECDN(2)) * CT ! RC Modified505 VPOL = (DECDN(1)-DECDN(2)) * CT
508 DECDD(1) = 0.5D0 * ( DECDN(1) + DECDN(2) + VPOL )506 DECDD(1) = 0.5D0 * ( DECDN(1) + DECDN(2) + VPOL )
509 DECDD(2) = 0.5D0 * ( DECDN(1) + DECDN(2) - VPOL )507 DECDD(2) = 0.5D0 * ( DECDN(1) + DECDN(2) - VPOL )
510 DECDD(3) = 0.5d0 * (DECDN(1)-DECDN(2)) * ST * CP ! RC Modified508 DECDD(3) = 0.5d0 * (DECDN(1)-DECDN(2)) * ST * CP
511 DECDD(4) =-0.5d0 * (DECDN(1)-DECDN(2)) * ST * SP ! RC Modified509 DECDD(4) =-0.5d0 * (DECDN(1)-DECDN(2)) * ST * SP
512C Gradient terms510C Gradient terms
513 DO 40 IX = 1,3511 DO 40 IX = 1,3
514 DEXDGD(IX,1) = DEXDGN(IX,1)*C2**2 + DEXDGN(IX,2)*S2**2512 DEXDGD(IX,1) = DEXDGN(IX,1)*C2**2 + DEXDGN(IX,2)*S2**2
515513
=== modified file 'version.info'
--- version.info 2017-06-30 14:45:53 +0000
+++ version.info 2017-06-30 18:01:40 +0000
@@ -1,2 +1,2 @@
1siesta-4.0--523--xc-21siesta-4.0--523--xc-2--np-1
22

Subscribers

People subscribed via source and target branches

to all changes: