--- trunk/OOPSE/libmdtools/calc_dipole_dipole.F90	2003/03/21 17:42:12	378
+++ trunk/OOPSE/libmdtools/calc_dipole_dipole.F90	2003/07/15 22:22:41	619
@@ -4,15 +4,16 @@ module dipole_dipole
   use definitions
   use atype_module
   use vector_class
+  use simulation
 #ifdef IS_MPI
   use mpiSimulation
 #endif
   implicit none
 
-  real(kind=dp), save :: rrf
-  real(kind=dp), save :: rt  
-  real(kind=dp), save :: rrfsq
-  real(kind=dp), save :: pre
+  real(kind=dp), save :: rrf = 0.0
+  real(kind=dp), save :: rt  = 0.0
+  real(kind=dp), save :: rrfsq = 0.0
+  real(kind=dp), save :: pre = 0.0
   logical, save :: dipole_initialized = .false.
 
 contains
@@ -23,8 +24,8 @@ contains
     rt = this_rt    
     rrfsq = rrf * rrf
     ! pre converts from mu in units of debye to kcal/mol
-    pre = 14.38362d0
-    
+    pre = 14.38362_dp
+
     dipole_initialized = .true.
     
     return
@@ -38,16 +39,16 @@ contains
 
     integer atom1, atom2, me1, me2
     real(kind=dp) :: rij, mu1, mu2
-    real(kind=dp) :: dfact1, dfact2, dip2, r2, r3, r5, pre
+    real(kind=dp) :: dfact1, dfact2, dip2, r2, r3, r5
     real(kind=dp) :: dudx, dudy, dudz, dudu1x, dudu1y, dudu1z
     real(kind=dp) :: dudu2x, dudu2y, dudu2z, rdotu1, rdotu2, u1dotu2
     real(kind=dp) :: taper, dtdr, vterm
 
     real( kind = dp ) :: pot
     real( kind = dp ), dimension(3) :: d
-    real( kind = dp ), dimension(:,:) :: u_l
-    real( kind = dp ), dimension(:,:) :: f
-    real( kind = dp ), dimension(:,:) :: t
+    real( kind = dp ), dimension(3,getNlocal()) :: u_l
+    real( kind = dp ), dimension(3,getNlocal()) :: f
+    real( kind = dp ), dimension(3,getNlocal()) :: t
     
     real (kind = dp), dimension(3) :: ul1
     real (kind = dp), dimension(3) :: ul2
@@ -79,11 +80,15 @@ contains
     ul2(3) = u_l(3,atom2)
 #endif
 
+
     call getElementProperty(atypes, me1, "dipole_moment", mu1)
     call getElementProperty(atypes, me2, "dipole_moment", mu2)
-    
+
+
     if (rij.le.rrf) then
+
        
+       
        if (rij.lt.rt) then
           taper = 1.0d0
           dtdr = 0.0d0
@@ -91,21 +96,20 @@ contains
           taper = (rrf + 2.0d0*rij - 3.0d0*rt)*(rrf-rij)**2/ ((rrf-rt)**3)
           dtdr = 6.0d0*(rij*rij - rij*rt - rij*rrf +rrf*rt)/((rrf-rt)**3)
        endif
-       
+
        r3 = r2*rij
        r5 = r3*r2
        
        rdotu1 = d(1)*ul1(1) + d(2)*ul1(2) + d(3)*ul1(3)
        rdotu2 = d(1)*ul2(1) + d(2)*ul2(2) + d(3)*ul2(3)
        u1dotu2 = ul1(1)*ul2(1) + ul1(2)*ul2(2) + ul1(3)*ul2(3)
-       
+
        dip2 = pre * mu1 * mu2
-       
        dfact1 = 3.0d0*dip2 / r2
        dfact2 = 3.0d0*dip2 / r5
-       
+
        vterm = dip2*((u1dotu2/r3) - 3.0d0*(rdotu1*rdotu2/r5))
-       
+
        if (do_pot) then
 #ifdef IS_MPI 
           pot_row(atom1) = pot_row(atom1) + 0.5d0*vterm*taper
@@ -129,16 +133,16 @@ contains
             (5.0d0*(rdotu1*rdotu2)/r5)) -  &
             dfact2*(ul1(3)*rdotu2 + ul2(3)*rdotu1))*taper +  &
             vterm*dtdr*d(3)/rij
-       
+
        dudu1x = (dip2*((ul2(1)/r3) - (3.0d0*d(1)*rdotu2/r5)))*taper
        dudu1y = (dip2*((ul2(2)/r3) - (3.0d0*d(2)*rdotu2/r5)))*taper
        dudu1z = (dip2*((ul2(3)/r3) - (3.0d0*d(3)*rdotu2/r5)))*taper
-       
+
        dudu2x = (dip2*((ul1(1)/r3) - (3.0d0*d(1)*rdotu1/r5)))*taper
        dudu2y = (dip2*((ul1(2)/r3) - (3.0d0*d(2)*rdotu1/r5)))*taper
        dudu2z = (dip2*((ul1(3)/r3) - (3.0d0*d(3)*rdotu1/r5)))*taper
        
-       
+
 #ifdef IS_MPI
        f_Row(1,atom1) = f_Row(1,atom1) + dudx
        f_Row(2,atom1) = f_Row(2,atom1) + dudy
@@ -174,16 +178,27 @@ contains
 #endif
        
        if (do_stress) then          
-          tau_Temp(1) = tau_Temp(1) + dudx * d(1)
-          tau_Temp(2) = tau_Temp(2) + dudx * d(2)
-          tau_Temp(3) = tau_Temp(3) + dudx * d(3)
-          tau_Temp(4) = tau_Temp(4) + dudy * d(1)
-          tau_Temp(5) = tau_Temp(5) + dudy * d(2)
-          tau_Temp(6) = tau_Temp(6) + dudy * d(3)
-          tau_Temp(7) = tau_Temp(7) + dudz * d(1)
-          tau_Temp(8) = tau_Temp(8) + dudz * d(2)
-          tau_Temp(9) = tau_Temp(9) + dudz * d(3)
-          virial_Temp = virial_Temp + (tau_Temp(1) + tau_Temp(5) + tau_Temp(9))
+
+          if (molMembershipList(atom1) .ne. molMembershipList(atom2)) then
+
+             ! because the d vector is the rj - ri vector, and 
+             ! because dudx, dudy, dudz are the (positive) force on
+             ! atom i (negative on atom j) we need a negative sign here:
+
+             tau_Temp(1) = tau_Temp(1) - d(1) * dudx
+             tau_Temp(2) = tau_Temp(2) - d(1) * dudy
+             tau_Temp(3) = tau_Temp(3) - d(1) * dudz
+             tau_Temp(4) = tau_Temp(4) - d(2) * dudx
+             tau_Temp(5) = tau_Temp(5) - d(2) * dudy
+             tau_Temp(6) = tau_Temp(6) - d(2) * dudz
+             tau_Temp(7) = tau_Temp(7) - d(3) * dudx
+             tau_Temp(8) = tau_Temp(8) - d(3) * dudy
+             tau_Temp(9) = tau_Temp(9) - d(3) * dudz
+
+             virial_Temp = virial_Temp + &
+                  (tau_Temp(1) + tau_Temp(5) + tau_Temp(9))
+
+          endif          
        endif
        
     endif