ViewVC Help
View File | Revision Log | Show Annotations | View Changeset | Root Listing
root/OpenMD/branches/development/src/nonbonded/Electrostatic.cpp
(Generate patch)

Comparing branches/development/src/nonbonded/Electrostatic.cpp (file contents):
Revision 1545 by gezelter, Fri Apr 8 21:25:19 2011 UTC vs.
Revision 1587 by gezelter, Fri Jul 8 20:25:32 2011 UTC

# Line 34 | Line 34
34   * work.  Good starting points are:
35   *                                                                      
36   * [1]  Meineke, et al., J. Comp. Chem. 26, 252-271 (2005).            
37 < * [2]  Fennell & Gezelter, J. Chem. Phys. 124, 234104 (2006).          
37 > * [2]  Fennell & Gezelter, J. Chem. Phys. 124 234104 (2006).          
38   * [3]  Sun, Lin & Gezelter, J. Chem. Phys. 128, 24107 (2008).          
39   * [4]  Vardeman & Gezelter, in progress (2009).                        
40   */
# Line 52 | Line 52 | namespace OpenMD {
52   namespace OpenMD {
53    
54    Electrostatic::Electrostatic(): name_("Electrostatic"), initialized_(false),
55 <                                  forceField_(NULL) {}
55 >                                  forceField_(NULL), info_(NULL),
56 >                                  haveCutoffRadius_(false),
57 >                                  haveDampingAlpha_(false),
58 >                                  haveDielectric_(false),
59 >                                  haveElectroSpline_(false)
60 >  {}
61    
62    void Electrostatic::initialize() {
63 +    
64 +    Globals* simParams_ = info_->getSimParams();
65  
59 –    Globals* simParams_;
60 –
66      summationMap_["HARD"]               = esm_HARD;
67      summationMap_["SWITCHING_FUNCTION"] = esm_SWITCHING_FUNCTION;
68      summationMap_["SHIFTED_POTENTIAL"]  = esm_SHIFTED_POTENTIAL;
# Line 97 | Line 102 | namespace OpenMD {
102      screeningMethod_ = UNDAMPED;
103      dielectric_ = 1.0;
104      one_third_ = 1.0 / 3.0;
100 –    haveCutoffRadius_ = false;
101 –    haveDampingAlpha_ = false;
102 –    haveDielectric_ = false;  
103 –    haveElectroSpline_ = false;
105    
106      // check the summation method:
107      if (simParams_->haveElectrostaticSummationMethod()) {
# Line 407 | Line 408 | namespace OpenMD {
408      return;
409    }
410    
411 <  void Electrostatic::setElectrostaticCutoffRadius( RealType theECR,
412 <                                                    RealType theRSW ) {
412 <    cutoffRadius_ = theECR;
411 >  void Electrostatic::setCutoffRadius( RealType rCut ) {
412 >    cutoffRadius_ = rCut;
413      rrf_ = cutoffRadius_;
414 –    rt_ = theRSW;
414      haveCutoffRadius_ = true;
415    }
416 +
417 +  void Electrostatic::setSwitchingRadius( RealType rSwitch ) {
418 +    rt_ = rSwitch;
419 +  }
420    void Electrostatic::setElectrostaticSummationMethod( ElectrostaticSummationMethod esm ) {
421      summationMethod_ = esm;
422    }
# Line 443 | Line 446 | namespace OpenMD {
446      RealType ct_i, ct_j, ct_ij, a1;
447      RealType riji, ri, ri2, ri3, ri4;
448      RealType pref, vterm, epot, dudr;
449 +    RealType vpair(0.0);
450      RealType scale, sc2;
451      RealType pot_term, preVal, rfVal;
452      RealType c2ri, c3ri, c4rij, cti3, ctj3, ctidotj;
453      RealType preSw, preSwSc;
454      RealType c1, c2, c3, c4;
455 <    RealType erfcVal, derfcVal;
455 >    RealType erfcVal(1.0), derfcVal(0.0);
456      RealType BigR;
457  
458      Vector3d Q_i, Q_j;
# Line 459 | Line 463 | namespace OpenMD {
463      Vector3d rhatdot2, rhatc4;
464      Vector3d dVdr;
465  
466 +    // variables for indirect (reaction field) interactions for excluded pairs:
467 +    RealType indirect_Pot(0.0);
468 +    RealType indirect_vpair(0.0);
469 +    Vector3d indirect_dVdr(V3Zero);
470 +    Vector3d indirect_duduz_i(V3Zero), indirect_duduz_j(V3Zero);
471 +
472      pair<RealType, RealType> res;
473      
474      if (!initialized_) initialize();
# Line 468 | Line 478 | namespace OpenMD {
478      
479      // some variables we'll need independent of electrostatic type:
480  
481 <    riji = 1.0 / idat.rij;
482 <    Vector3d rhat = idat.d  * riji;
481 >    riji = 1.0 /  *(idat.rij) ;
482 >    Vector3d rhat =  *(idat.d)   * riji;
483  
484      // logicals
485  
# Line 483 | Line 493 | namespace OpenMD {
493      bool j_is_SplitDipole = data2.is_SplitDipole;
494      bool j_is_Quadrupole = data2.is_Quadrupole;
495      
496 <    if (i_is_Charge)
496 >    if (i_is_Charge) {
497        q_i = data1.charge;
498 +      if (idat.excluded) {
499 +        *(idat.skippedCharge2) += q_i;
500 +      }
501 +    }
502  
503      if (i_is_Dipole) {
504        mu_i = data1.dipole_moment;
505 <      uz_i = idat.eFrame1.getColumn(2);
505 >      uz_i = idat.eFrame1->getColumn(2);
506        
507        ct_i = dot(uz_i, rhat);
508  
# Line 504 | Line 518 | namespace OpenMD {
518        qyy_i = Q_i.y();
519        qzz_i = Q_i.z();
520        
521 <      ux_i = idat.eFrame1.getColumn(0);
522 <      uy_i = idat.eFrame1.getColumn(1);
523 <      uz_i = idat.eFrame1.getColumn(2);
521 >      ux_i = idat.eFrame1->getColumn(0);
522 >      uy_i = idat.eFrame1->getColumn(1);
523 >      uz_i = idat.eFrame1->getColumn(2);
524  
525        cx_i = dot(ux_i, rhat);
526        cy_i = dot(uy_i, rhat);
# Line 517 | Line 531 | namespace OpenMD {
531        duduz_i = V3Zero;
532      }
533  
534 <    if (j_is_Charge)
534 >    if (j_is_Charge) {
535        q_j = data2.charge;
536 +      if (idat.excluded) {
537 +        *(idat.skippedCharge1) += q_j;
538 +      }
539 +    }
540  
541 +
542      if (j_is_Dipole) {
543        mu_j = data2.dipole_moment;
544 <      uz_j = idat.eFrame2.getColumn(2);
544 >      uz_j = idat.eFrame2->getColumn(2);
545        
546        ct_j = dot(uz_j, rhat);
547  
# Line 538 | Line 557 | namespace OpenMD {
557        qyy_j = Q_j.y();
558        qzz_j = Q_j.z();
559        
560 <      ux_j = idat.eFrame2.getColumn(0);
561 <      uy_j = idat.eFrame2.getColumn(1);
562 <      uz_j = idat.eFrame2.getColumn(2);
560 >      ux_j = idat.eFrame2->getColumn(0);
561 >      uy_j = idat.eFrame2->getColumn(1);
562 >      uz_j = idat.eFrame2->getColumn(2);
563  
564        cx_j = dot(ux_j, rhat);
565        cy_j = dot(uy_j, rhat);
# Line 559 | Line 578 | namespace OpenMD {
578        if (j_is_Charge) {
579          if (screeningMethod_ == DAMPED) {
580            // assemble the damping variables
581 <          res = erfcSpline_->getValueAndDerivativeAt(idat.rij);
581 >          res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
582            erfcVal = res.first;
583            derfcVal = res.second;
584            c1 = erfcVal * riji;
# Line 569 | Line 588 | namespace OpenMD {
588            c2 = c1 * riji;
589          }
590  
591 <        preVal = idat.electroMult * pre11_ * q_i * q_j;
591 >        preVal =  *(idat.electroMult) * pre11_ * q_i * q_j;
592          
593          if (summationMethod_ == esm_SHIFTED_POTENTIAL) {
594            vterm = preVal * (c1 - c1c_);
595 <          dudr  = -idat.sw * preVal * c2;
595 >          dudr  = - *(idat.sw)  * preVal * c2;
596  
597          } else if (summationMethod_ == esm_SHIFTED_FORCE)  {
598 <          vterm = preVal * ( c1 - c1c_ + c2c_*(idat.rij - cutoffRadius_) );
599 <          dudr  = idat.sw * preVal * (c2c_ - c2);
598 >          vterm = preVal * ( c1 - c1c_ + c2c_*( *(idat.rij)  - cutoffRadius_) );
599 >          dudr  =  *(idat.sw)  * preVal * (c2c_ - c2);
600  
601          } else if (summationMethod_ == esm_REACTION_FIELD) {
602 <          rfVal = idat.electroMult * preRF_ * idat.rij * idat.rij;
602 >          rfVal = preRF_ *  *(idat.rij)  *  *(idat.rij);
603 >
604            vterm = preVal * ( riji + rfVal );            
605 <          dudr  = idat.sw * preVal * ( 2.0 * rfVal - riji ) * riji;
605 >          dudr  =  *(idat.sw)  * preVal * ( 2.0 * rfVal - riji ) * riji;
606 >          
607 >          // if this is an excluded pair, there are still indirect
608 >          // interactions via the reaction field we must worry about:
609  
610 +          if (idat.excluded) {
611 +            indirect_vpair += preVal * rfVal;
612 +            indirect_Pot += *(idat.sw) * preVal * rfVal;
613 +            indirect_dVdr += *(idat.sw)  * preVal * 2.0 * rfVal  * riji * rhat;
614 +          }
615 +          
616          } else {
588 –          vterm = preVal * riji * erfcVal;            
617  
618 <          dudr  = - idat.sw * preVal * c2;
618 >          vterm = preVal * riji * erfcVal;          
619 >          dudr  = -  *(idat.sw)  * preVal * c2;
620  
621          }
593 –
594 –        idat.vpair[2] += vterm;
595 –        epot += idat.sw * vterm;
622  
623 <        dVdr += dudr * rhat;      
623 >        vpair += vterm;
624 >        epot +=  *(idat.sw)  * vterm;
625 >        dVdr += dudr * rhat;                
626        }
627  
628        if (j_is_Dipole) {
629          // pref is used by all the possible methods
630 <        pref = idat.electroMult * pre12_ * q_i * mu_j;
631 <        preSw = idat.sw * pref;
630 >        pref =  *(idat.electroMult) * pre12_ * q_i * mu_j;
631 >        preSw =  *(idat.sw)  * pref;
632  
633          if (summationMethod_ == esm_REACTION_FIELD) {
634            ri2 = riji * riji;
635            ri3 = ri2 * riji;
636      
637 <          vterm = - pref * ct_j * ( ri2 - preRF2_ * idat.rij );
638 <          idat.vpair[2] += vterm;
639 <          epot += idat.sw * vterm;
637 >          vterm = - pref * ct_j * ( ri2 - preRF2_ *  *(idat.rij)  );
638 >          vpair += vterm;
639 >          epot +=  *(idat.sw)  * vterm;
640  
641            dVdr +=  -preSw * (ri3 * (uz_j - 3.0 * ct_j * rhat) - preRF2_*uz_j);
642 <          duduz_j += -preSw * rhat * (ri2 - preRF2_ * idat.rij);  
642 >          duduz_j += -preSw * rhat * (ri2 - preRF2_ *  *(idat.rij) );  
643  
644 +          // Even if we excluded this pair from direct interactions,
645 +          // we still have the reaction-field-mediated charge-dipole
646 +          // interaction:
647 +
648 +          if (idat.excluded) {
649 +            indirect_vpair += pref * ct_j * preRF2_ * *(idat.rij);
650 +            indirect_Pot += preSw * ct_j * preRF2_ * *(idat.rij);
651 +            indirect_dVdr += preSw * preRF2_ * uz_j;
652 +            indirect_duduz_j += preSw * rhat * preRF2_ *  *(idat.rij);
653 +          }
654 +                      
655          } else {
656            // determine the inverse r used if we have split dipoles
657            if (j_is_SplitDipole) {
658 <            BigR = sqrt(idat.r2 + 0.25 * d_j * d_j);
658 >            BigR = sqrt( *(idat.r2) + 0.25 * d_j * d_j);
659              ri = 1.0 / BigR;
660 <            scale = idat.rij * ri;
660 >            scale =  *(idat.rij)  * ri;
661            } else {
662              ri = riji;
663              scale = 1.0;
# Line 628 | Line 667 | namespace OpenMD {
667  
668            if (screeningMethod_ == DAMPED) {
669              // assemble the damping variables
670 <            res = erfcSpline_->getValueAndDerivativeAt(idat.rij);
670 >            res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
671              erfcVal = res.first;
672              derfcVal = res.second;
673              c1 = erfcVal * ri;
# Line 645 | Line 684 | namespace OpenMD {
684            // calculate the potential
685            pot_term =  scale * c2;
686            vterm = -pref * ct_j * pot_term;
687 <          idat.vpair[2] += vterm;
688 <          epot += idat.sw * vterm;
687 >          vpair += vterm;
688 >          epot +=  *(idat.sw)  * vterm;
689              
690            // calculate derivatives for forces and torques
691  
# Line 661 | Line 700 | namespace OpenMD {
700          cx2 = cx_j * cx_j;
701          cy2 = cy_j * cy_j;
702          cz2 = cz_j * cz_j;
703 <        pref =  idat.electroMult * pre14_ * q_i * one_third_;
703 >        pref =   *(idat.electroMult) * pre14_ * q_i * one_third_;
704            
705          if (screeningMethod_ == DAMPED) {
706            // assemble the damping variables
707 <          res = erfcSpline_->getValueAndDerivativeAt(idat.rij);
707 >          res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
708            erfcVal = res.first;
709            derfcVal = res.second;
710            c1 = erfcVal * riji;
# Line 680 | Line 719 | namespace OpenMD {
719          }
720  
721          // precompute variables for convenience
722 <        preSw = idat.sw * pref;
722 >        preSw =  *(idat.sw)  * pref;
723          c2ri = c2 * riji;
724          c3ri = c3 * riji;
725 <        c4rij = c4 * idat.rij;
725 >        c4rij = c4 *  *(idat.rij) ;
726          rhatdot2 = 2.0 * rhat * c3;
727          rhatc4 = rhat * c4rij;
728  
# Line 692 | Line 731 | namespace OpenMD {
731                       qyy_j * (cy2*c3 - c2ri) +
732                       qzz_j * (cz2*c3 - c2ri) );
733          vterm = pref * pot_term;
734 <        idat.vpair[2] += vterm;
735 <        epot += idat.sw * vterm;
734 >        vpair += vterm;
735 >        epot +=  *(idat.sw)  * vterm;
736                  
737          // calculate derivatives for the forces and torques
738  
# Line 711 | Line 750 | namespace OpenMD {
750  
751        if (j_is_Charge) {
752          // variables used by all the methods
753 <        pref = idat.electroMult * pre12_ * q_j * mu_i;
754 <        preSw = idat.sw * pref;
753 >        pref =  *(idat.electroMult) * pre12_ * q_j * mu_i;
754 >        preSw =  *(idat.sw)  * pref;
755  
756          if (summationMethod_ == esm_REACTION_FIELD) {
757  
758            ri2 = riji * riji;
759            ri3 = ri2 * riji;
760  
761 <          vterm = pref * ct_i * ( ri2 - preRF2_ * idat.rij );
762 <          idat.vpair[2] += vterm;
763 <          epot += idat.sw * vterm;
761 >          vterm = pref * ct_i * ( ri2 - preRF2_ *  *(idat.rij)  );
762 >          vpair += vterm;
763 >          epot +=  *(idat.sw)  * vterm;
764            
765            dVdr += preSw * (ri3 * (uz_i - 3.0 * ct_i * rhat) - preRF2_ * uz_i);
766            
767 <          duduz_i += preSw * rhat * (ri2 - preRF2_ * idat.rij);
767 >          duduz_i += preSw * rhat * (ri2 - preRF2_ *  *(idat.rij) );
768 >
769 >          // Even if we excluded this pair from direct interactions,
770 >          // we still have the reaction-field-mediated charge-dipole
771 >          // interaction:
772 >
773 >          if (idat.excluded) {
774 >            indirect_vpair += -pref * ct_i * preRF2_ * *(idat.rij);
775 >            indirect_Pot += -preSw * ct_i * preRF2_ * *(idat.rij);
776 >            indirect_dVdr += -preSw * preRF2_ * uz_i;
777 >            indirect_duduz_i += -preSw * rhat * preRF2_ *  *(idat.rij);
778 >          }
779              
780          } else {
781            
782            // determine inverse r if we are using split dipoles
783            if (i_is_SplitDipole) {
784 <            BigR = sqrt(idat.r2 + 0.25 * d_i * d_i);
784 >            BigR = sqrt( *(idat.r2) + 0.25 * d_i * d_i);
785              ri = 1.0 / BigR;
786 <            scale = idat.rij * ri;
786 >            scale =  *(idat.rij)  * ri;
787            } else {
788              ri = riji;
789              scale = 1.0;
# Line 743 | Line 793 | namespace OpenMD {
793              
794            if (screeningMethod_ == DAMPED) {
795              // assemble the damping variables
796 <            res = erfcSpline_->getValueAndDerivativeAt(idat.rij);
796 >            res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
797              erfcVal = res.first;
798              derfcVal = res.second;
799              c1 = erfcVal * ri;
# Line 760 | Line 810 | namespace OpenMD {
810            // calculate the potential
811            pot_term = c2 * scale;
812            vterm = pref * ct_i * pot_term;
813 <          idat.vpair[2] += vterm;
814 <          epot += idat.sw * vterm;
813 >          vpair += vterm;
814 >          epot +=  *(idat.sw)  * vterm;
815  
816            // calculate derivatives for the forces and torques
817            dVdr += preSw * (uz_i * c2ri - ct_i * rhat * sc2 * c3);
# Line 773 | Line 823 | namespace OpenMD {
823          // variables used by all methods
824          ct_ij = dot(uz_i, uz_j);
825  
826 <        pref = idat.electroMult * pre22_ * mu_i * mu_j;
827 <        preSw = idat.sw * pref;
826 >        pref =  *(idat.electroMult) * pre22_ * mu_i * mu_j;
827 >        preSw =  *(idat.sw)  * pref;
828  
829          if (summationMethod_ == esm_REACTION_FIELD) {
830            ri2 = riji * riji;
# Line 783 | Line 833 | namespace OpenMD {
833  
834            vterm = pref * ( ri3 * (ct_ij - 3.0 * ct_i * ct_j) -
835                             preRF2_ * ct_ij );
836 <          idat.vpair[2] += vterm;
837 <          epot += idat.sw * vterm;
836 >          vpair += vterm;
837 >          epot +=  *(idat.sw)  * vterm;
838              
839            a1 = 5.0 * ct_i * ct_j - ct_ij;
840              
# Line 793 | Line 843 | namespace OpenMD {
843            duduz_i += preSw * (ri3 * (uz_j - 3.0 * ct_j * rhat) - preRF2_*uz_j);
844            duduz_j += preSw * (ri3 * (uz_i - 3.0 * ct_i * rhat) - preRF2_*uz_i);
845  
846 +          if (idat.excluded) {
847 +            indirect_vpair +=  - pref * preRF2_ * ct_ij;
848 +            indirect_Pot +=    - preSw * preRF2_ * ct_ij;
849 +            indirect_duduz_i += -preSw * preRF2_ * uz_j;
850 +            indirect_duduz_j += -preSw * preRF2_ * uz_i;
851 +          }
852 +
853          } else {
854            
855            if (i_is_SplitDipole) {
856              if (j_is_SplitDipole) {
857 <              BigR = sqrt(idat.r2 + 0.25 * d_i * d_i + 0.25 * d_j * d_j);
857 >              BigR = sqrt( *(idat.r2) + 0.25 * d_i * d_i + 0.25 * d_j * d_j);
858              } else {
859 <              BigR = sqrt(idat.r2 + 0.25 * d_i * d_i);
859 >              BigR = sqrt( *(idat.r2) + 0.25 * d_i * d_i);
860              }
861              ri = 1.0 / BigR;
862 <            scale = idat.rij * ri;
862 >            scale =  *(idat.rij)  * ri;
863            } else {
864              if (j_is_SplitDipole) {
865 <              BigR = sqrt(idat.r2 + 0.25 * d_j * d_j);
865 >              BigR = sqrt( *(idat.r2) + 0.25 * d_j * d_j);
866                ri = 1.0 / BigR;
867 <              scale = idat.rij * ri;
867 >              scale =  *(idat.rij)  * ri;
868              } else {
869                ri = riji;
870                scale = 1.0;
# Line 815 | Line 872 | namespace OpenMD {
872            }
873            if (screeningMethod_ == DAMPED) {
874              // assemble damping variables
875 <            res = erfcSpline_->getValueAndDerivativeAt(idat.rij);
875 >            res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
876              erfcVal = res.first;
877              derfcVal = res.second;
878              c1 = erfcVal * ri;
# Line 837 | Line 894 | namespace OpenMD {
894            preSwSc = preSw * scale;
895            c2ri = c2 * ri;
896            c3ri = c3 * ri;
897 <          c4rij = c4 * idat.rij;
897 >          c4rij = c4 *  *(idat.rij) ;
898  
899            // calculate the potential
900            pot_term = (ct_ij * c2ri - ctidotj * c3);
901            vterm = pref * pot_term;
902 <          idat.vpair[2] += vterm;
903 <          epot += idat.sw * vterm;
902 >          vpair += vterm;
903 >          epot +=  *(idat.sw)  * vterm;
904  
905            // calculate derivatives for the forces and torques
906            dVdr += preSwSc * ( ctidotj * rhat * c4rij  -
# Line 862 | Line 919 | namespace OpenMD {
919          cy2 = cy_i * cy_i;
920          cz2 = cz_i * cz_i;
921  
922 <        pref = idat.electroMult * pre14_ * q_j * one_third_;
922 >        pref =  *(idat.electroMult) * pre14_ * q_j * one_third_;
923  
924          if (screeningMethod_ == DAMPED) {
925            // assemble the damping variables
926 <          res = erfcSpline_->getValueAndDerivativeAt(idat.rij);
926 >          res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
927            erfcVal = res.first;
928            derfcVal = res.second;
929            c1 = erfcVal * riji;
# Line 881 | Line 938 | namespace OpenMD {
938          }
939            
940          // precompute some variables for convenience
941 <        preSw = idat.sw * pref;
941 >        preSw =  *(idat.sw)  * pref;
942          c2ri = c2 * riji;
943          c3ri = c3 * riji;
944 <        c4rij = c4 * idat.rij;
944 >        c4rij = c4 *  *(idat.rij) ;
945          rhatdot2 = 2.0 * rhat * c3;
946          rhatc4 = rhat * c4rij;
947  
# Line 894 | Line 951 | namespace OpenMD {
951                       qzz_i * (cz2 * c3 - c2ri) );
952          
953          vterm = pref * pot_term;
954 <        idat.vpair[2] += vterm;
955 <        epot += idat.sw * vterm;
954 >        vpair += vterm;
955 >        epot +=  *(idat.sw)  * vterm;
956  
957          // calculate the derivatives for the forces and torques
958  
# Line 909 | Line 966 | namespace OpenMD {
966        }
967      }
968  
912 –    idat.pot[2] += epot;
913 –    idat.f1 += dVdr;
969  
970 <    if (i_is_Dipole || i_is_Quadrupole)
971 <      idat.t1 -= cross(uz_i, duduz_i);
972 <    if (i_is_Quadrupole) {
973 <      idat.t1 -= cross(ux_i, dudux_i);
974 <      idat.t1 -= cross(uy_i, duduy_i);
975 <    }
970 >    if (!idat.excluded) {
971 >      *(idat.vpair) += vpair;
972 >      (*(idat.pot))[ELECTROSTATIC_FAMILY] += epot;
973 >      *(idat.f1) += dVdr;
974 >      
975 >      if (i_is_Dipole || i_is_Quadrupole)
976 >        *(idat.t1) -= cross(uz_i, duduz_i);
977 >      if (i_is_Quadrupole) {
978 >        *(idat.t1) -= cross(ux_i, dudux_i);
979 >        *(idat.t1) -= cross(uy_i, duduy_i);
980 >      }
981 >      
982 >      if (j_is_Dipole || j_is_Quadrupole)
983 >        *(idat.t2) -= cross(uz_j, duduz_j);
984 >      if (j_is_Quadrupole) {
985 >        *(idat.t2) -= cross(uz_j, dudux_j);
986 >        *(idat.t2) -= cross(uz_j, duduy_j);
987 >      }
988  
989 <    if (j_is_Dipole || j_is_Quadrupole)
990 <      idat.t2 -= cross(uz_j, duduz_j);
991 <    if (j_is_Quadrupole) {
992 <      idat.t2 -= cross(uz_j, dudux_j);
993 <      idat.t2 -= cross(uz_j, duduy_j);
989 >    } else {
990 >
991 >      // only accumulate the forces and torques resulting from the
992 >      // indirect reaction field terms.
993 >      *(idat.vpair) += indirect_vpair;
994 >      (*(idat.pot))[ELECTROSTATIC_FAMILY] += indirect_Pot;
995 >      *(idat.f1) += indirect_dVdr;
996 >      
997 >      if (i_is_Dipole)
998 >        *(idat.t1) -= cross(uz_i, indirect_duduz_i);
999 >      if (j_is_Dipole)
1000 >        *(idat.t2) -= cross(uz_j, indirect_duduz_j);
1001      }
1002  
1003 +
1004      return;
1005    }  
1006  
# Line 945 | Line 1020 | namespace OpenMD {
1020      bool j_is_Dipole = data2.is_Dipole;
1021  
1022      RealType q_i, q_j;
948 –    
949 –    // The skippedCharge computation is needed by the real-space cutoff methods
950 –    // (i.e. shifted force and shifted potential)
1023  
1024 +    // The skippedCharge computation is needed by the real-space
1025 +    // cutoff methods (i.e. shifted force and shifted potential)
1026 +      
1027      if (i_is_Charge) {
1028        q_i = data1.charge;
1029 <      idat.skippedCharge2 += q_i;
1029 >      *(idat.skippedCharge2) += q_i;
1030      }
1031 <
1031 >    
1032      if (j_is_Charge) {
1033        q_j = data2.charge;
1034 <      idat.skippedCharge1 += q_j;
1034 >      *(idat.skippedCharge1) += q_j;
1035      }
1036  
1037      // the rest of this function should only be necessary for reaction field.
# Line 970 | Line 1045 | namespace OpenMD {
1045  
1046        // some variables we'll need independent of electrostatic type:
1047        
1048 <      riji = 1.0 / idat.rij;
1049 <      rhat = idat.d  * riji;
1048 >      riji = 1.0 /  *(idat.rij) ;
1049 >      rhat =  *(idat.d)  * riji;
1050  
1051        if (i_is_Dipole) {
1052          mu_i = data1.dipole_moment;
1053 <        uz_i = idat.eFrame1.getColumn(2);      
1053 >        uz_i = idat.eFrame1->getColumn(2);      
1054          ct_i = dot(uz_i, rhat);
1055          duduz_i = V3Zero;
1056        }
1057              
1058        if (j_is_Dipole) {
1059          mu_j = data2.dipole_moment;
1060 <        uz_j = idat.eFrame2.getColumn(2);      
1060 >        uz_j = idat.eFrame2->getColumn(2);      
1061          ct_j = dot(uz_j, rhat);
1062          duduz_j = V3Zero;
1063        }
1064      
1065        if (i_is_Charge) {
1066          if (j_is_Charge) {
1067 <          preVal = idat.electroMult * pre11_ * q_i * q_j;
1068 <          rfVal = preRF_ * idat.rij * idat.rij;
1067 >          preVal =  *(idat.electroMult) * pre11_ * q_i * q_j;
1068 >          rfVal = preRF_ *  *(idat.rij)  *  *(idat.rij) ;
1069            vterm = preVal * rfVal;
1070 <          myPot += idat.sw * vterm;        
1071 <          dudr  = idat.sw * preVal * 2.0 * rfVal * riji;        
1070 >          myPot +=  *(idat.sw)  * vterm;        
1071 >          dudr  =  *(idat.sw)  * preVal * 2.0 * rfVal * riji;        
1072            dVdr += dudr * rhat;
1073          }
1074          
1075          if (j_is_Dipole) {
1076            ri2 = riji * riji;
1077            ri3 = ri2 * riji;        
1078 <          pref = idat.electroMult * pre12_ * q_i * mu_j;
1079 <          vterm = - pref * ct_j * ( ri2 - preRF2_ * idat.rij );
1080 <          myPot += idat.sw * vterm;        
1081 <          dVdr += -idat.sw * pref * ( ri3 * ( uz_j - 3.0 * ct_j * rhat) - preRF2_ * uz_j);
1082 <          duduz_j += -idat.sw * pref * rhat * (ri2 - preRF2_ * idat.rij);
1078 >          pref =  *(idat.electroMult) * pre12_ * q_i * mu_j;
1079 >          vterm = - pref * ct_j * ( ri2 - preRF2_ *  *(idat.rij)  );
1080 >          myPot +=  *(idat.sw)  * vterm;        
1081 >          dVdr += - *(idat.sw)  * pref * ( ri3 * ( uz_j - 3.0 * ct_j * rhat) - preRF2_ * uz_j);
1082 >          duduz_j += - *(idat.sw)  * pref * rhat * (ri2 - preRF2_ *  *(idat.rij) );
1083          }
1084        }
1085        if (i_is_Dipole) {
1086          if (j_is_Charge) {
1087            ri2 = riji * riji;
1088            ri3 = ri2 * riji;        
1089 <          pref = idat.electroMult * pre12_ * q_j * mu_i;
1090 <          vterm = - pref * ct_i * ( ri2 - preRF2_ * idat.rij );
1091 <          myPot += idat.sw * vterm;        
1092 <          dVdr += idat.sw * pref * ( ri3 * ( uz_i - 3.0 * ct_i * rhat) - preRF2_ * uz_i);      
1093 <          duduz_i += idat.sw * pref * rhat * (ri2 - preRF2_ * idat.rij);
1089 >          pref =  *(idat.electroMult) * pre12_ * q_j * mu_i;
1090 >          vterm = - pref * ct_i * ( ri2 - preRF2_ *  *(idat.rij)  );
1091 >          myPot +=  *(idat.sw)  * vterm;        
1092 >          dVdr +=  *(idat.sw)  * pref * ( ri3 * ( uz_i - 3.0 * ct_i * rhat) - preRF2_ * uz_i);      
1093 >          duduz_i +=  *(idat.sw)  * pref * rhat * (ri2 - preRF2_ *  *(idat.rij));
1094          }
1095        }
1096        
1097        // accumulate the forces and torques resulting from the self term
1098 <      idat.pot[2] += myPot;
1099 <      idat.f1 += dVdr;
1098 >      (*(idat.pot))[ELECTROSTATIC_FAMILY] += myPot;
1099 >      *(idat.f1) += dVdr;
1100        
1101        if (i_is_Dipole)
1102 <        idat.t1 -= cross(uz_i, duduz_i);
1102 >        *(idat.t1) -= cross(uz_i, duduz_i);
1103        if (j_is_Dipole)
1104 <        idat.t2 -= cross(uz_j, duduz_j);
1104 >        *(idat.t2) -= cross(uz_j, duduz_j);
1105      }
1106    }
1107      
# Line 1034 | Line 1109 | namespace OpenMD {
1109      RealType mu1, preVal, chg1, self;
1110      
1111      if (!initialized_) initialize();
1112 <    
1112 >
1113      ElectrostaticAtomData data = ElectrostaticMap[sdat.atype];
1114    
1115      // logicals
1041 –
1116      bool i_is_Charge = data.is_Charge;
1117      bool i_is_Dipole = data.is_Dipole;
1118  
# Line 1046 | Line 1120 | namespace OpenMD {
1120        if (i_is_Dipole) {
1121          mu1 = data.dipole_moment;          
1122          preVal = pre22_ * preRF2_ * mu1 * mu1;
1123 <        sdat.pot[2] -= 0.5 * preVal;
1123 >        (*(sdat.pot))[ELECTROSTATIC_FAMILY] -= 0.5 * preVal;
1124          
1125          // The self-correction term adds into the reaction field vector
1126 <        Vector3d uz_i = sdat.eFrame.getColumn(2);
1126 >        Vector3d uz_i = sdat.eFrame->getColumn(2);
1127          Vector3d ei = preVal * uz_i;
1128  
1129          // This looks very wrong.  A vector crossed with itself is zero.
1130 <        sdat.t -= cross(uz_i, ei);
1130 >        *(sdat.t) -= cross(uz_i, ei);
1131        }
1132      } else if (summationMethod_ == esm_SHIFTED_FORCE || summationMethod_ == esm_SHIFTED_POTENTIAL) {
1133        if (i_is_Charge) {        
1134          chg1 = data.charge;
1135          if (screeningMethod_ == DAMPED) {
1136 <          self = - 0.5 * (c1c_ + alphaPi_) * chg1 * (chg1 + sdat.skippedCharge) * pre11_;
1136 >          self = - 0.5 * (c1c_ + alphaPi_) * chg1 * (chg1 + *(sdat.skippedCharge)) * pre11_;
1137          } else {        
1138 <          self = - 0.5 * rcuti_ * chg1 * (chg1 + sdat.skippedCharge) * pre11_;
1138 >          self = - 0.5 * rcuti_ * chg1 * (chg1 +  *(sdat.skippedCharge)) * pre11_;
1139          }
1140 <        sdat.pot[2] += self;
1140 >        (*(sdat.pot))[ELECTROSTATIC_FAMILY] += self;
1141        }
1142      }
1143    }

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines