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 1710 by gezelter, Fri May 18 21:44:02 2012 UTC vs.
Revision 1822 by gezelter, Tue Jan 8 15:29:03 2013 UTC

# Line 48 | Line 48
48   #include "utils/simError.h"
49   #include "types/NonBondedInteractionType.hpp"
50   #include "types/FixedChargeAdapter.hpp"
51 + #include "types/FluctuatingChargeAdapter.hpp"
52   #include "types/MultipoleAdapter.hpp"
53   #include "io/Globals.hpp"
54 + #include "nonbonded/SlaterIntegrals.hpp"
55 + #include "utils/PhysicalConstants.hpp"
56 + #include "math/erfc.hpp"
57 + #include "math/SquareMatrix.hpp"
58  
59   namespace OpenMD {
60    
# Line 58 | Line 63 | namespace OpenMD {
63                                    haveCutoffRadius_(false),
64                                    haveDampingAlpha_(false),
65                                    haveDielectric_(false),
66 <                                  haveElectroSpline_(false)
66 >                                  haveElectroSplines_(false)
67    {}
68    
69    void Electrostatic::initialize() {
# Line 84 | Line 89 | namespace OpenMD {
89      // Charge-Dipole, assuming charges are measured in electrons, and
90      // dipoles are measured in debyes
91      pre12_ = 69.13373;
92 <    // Dipole-Dipole, assuming dipoles are measured in debyes
92 >    // Dipole-Dipole, assuming dipoles are measured in Debye
93      pre22_ = 14.39325;
94      // Charge-Quadrupole, assuming charges are measured in electrons, and
95      // quadrupoles are measured in 10^-26 esu cm^2
96 <    // This unit is also known affectionately as an esu centi-barn.
96 >    // This unit is also known affectionately as an esu centibarn.
97      pre14_ = 69.13373;
98 <    
98 >    // Dipole-Quadrupole, assuming dipoles are measured in debyes and
99 >    // quadrupoles in esu centibarns:
100 >    pre24_ = 14.39325;
101 >    // Quadrupole-Quadrupole, assuming esu centibarns:
102 >    pre44_ = 14.39325;
103 >
104      // conversions for the simulation box dipole moment
105      chargeToC_ = 1.60217733e-19;
106      angstromToM_ = 1.0e-10;
107      debyeToCm_ = 3.33564095198e-30;
108      
109      // number of points for electrostatic splines
110 <    np_ = 100;
110 >    np_ = 1000;
111      
112      // variables to handle different summation methods for long-range
113      // electrostatics:
114      summationMethod_ = esm_HARD;    
115      screeningMethod_ = UNDAMPED;
116      dielectric_ = 1.0;
107    one_third_ = 1.0 / 3.0;
117    
118      // check the summation method:
119      if (simParams_->haveElectrostaticSummationMethod()) {
# Line 189 | Line 198 | namespace OpenMD {
198          
199          // throw warning
200          sprintf( painCave.errMsg,
201 <                 "Electrostatic::initialize: dampingAlpha was not specified in the input file.\n"
202 <                 "\tA default value of %f (1/ang) will be used for the cutoff of\n\t%f (ang).\n",
201 >                 "Electrostatic::initialize: dampingAlpha was not specified in the\n"
202 >                 "\tinput file.  A default value of %f (1/ang) will be used for the\n"
203 >                 "\tcutoff of %f (ang).\n",
204                   dampingAlpha_, cutoffRadius_);
205          painCave.severity = OPENMD_INFO;
206          painCave.isFatal = 0;
# Line 211 | Line 221 | namespace OpenMD {
221        
222        if (at->isElectrostatic())
223          addType(at);
224 +    }  
225 +    
226 +    if (summationMethod_ == esm_REACTION_FIELD) {
227 +      preRF_ = (dielectric_ - 1.0) /
228 +        ((2.0 * dielectric_ + 1.0) * pow(cutoffRadius_,3) );
229      }
230      
231 +    RealType b0c, b1c, b2c, b3c, b4c, b5c;
232 +    RealType db0c_1, db0c_2, db0c_3, db0c_4, db0c_5;
233 +    RealType a2, expTerm, invArootPi;
234 +    
235 +    RealType r = cutoffRadius_;
236 +    RealType r2 = r * r;
237  
238 <    cutoffRadius2_ = cutoffRadius_ * cutoffRadius_;
239 <    rcuti_ = 1.0 / cutoffRadius_;
240 <    rcuti2_ = rcuti_ * rcuti_;
241 <    rcuti3_ = rcuti2_ * rcuti_;
242 <    rcuti4_ = rcuti2_ * rcuti2_;
243 <
244 <    if (screeningMethod_ == DAMPED) {
245 <      
246 <      alpha2_ = dampingAlpha_ * dampingAlpha_;
247 <      alpha4_ = alpha2_ * alpha2_;
248 <      alpha6_ = alpha4_ * alpha2_;
249 <      alpha8_ = alpha4_ * alpha4_;
229 <      
230 <      constEXP_ = exp(-alpha2_ * cutoffRadius2_);
231 <      invRootPi_ = 0.56418958354775628695;
232 <      alphaPi_ = 2.0 * dampingAlpha_ * invRootPi_;
233 <
234 <      c1c_ = erfc(dampingAlpha_ * cutoffRadius_) * rcuti_;
235 <      c2c_ = alphaPi_ * constEXP_ * rcuti_ + c1c_ * rcuti_;
236 <      c3c_ = 2.0 * alphaPi_ * alpha2_ + 3.0 * c2c_ * rcuti_;
237 <      c4c_ = 4.0 * alphaPi_ * alpha4_ + 5.0 * c3c_ * rcuti2_;
238 <      c5c_ = 8.0 * alphaPi_ * alpha6_ + 7.0 * c4c_ * rcuti2_;
239 <      c6c_ = 16.0 * alphaPi_ * alpha8_ + 9.0 * c5c_ * rcuti2_;
238 >    if (screeningMethod_ == DAMPED) {      
239 >      a2 = dampingAlpha_ * dampingAlpha_;
240 >      invArootPi = 1.0 / (dampingAlpha_ * sqrt(M_PI));    
241 >      expTerm = exp(-a2 * r2);
242 >      // values of Smith's B_l functions at the cutoff radius:
243 >      b0c = erfc(dampingAlpha_ * r) / r;
244 >      b1c = (      b0c     + 2.0*a2     * expTerm * invArootPi) / r2;
245 >      b2c = (3.0 * b1c + pow(2.0*a2, 2) * expTerm * invArootPi) / r2;
246 >      b3c = (5.0 * b2c + pow(2.0*a2, 3) * expTerm * invArootPi) / r2;
247 >      b4c = (7.0 * b3c + pow(2.0*a2, 4) * expTerm * invArootPi) / r2;
248 >      b5c = (9.0 * b4c + pow(2.0*a2, 5) * expTerm * invArootPi) / r2;
249 >      selfMult_ = b0c  +  2.0 * a2 * invArootPi;
250      } else {
251 <      c1c_ = rcuti_;
252 <      c2c_ = c1c_ * rcuti_;
253 <      c3c_ = 3.0 * c2c_ * rcuti_;
254 <      c4c_ = 5.0 * c3c_ * rcuti2_;
255 <      c5c_ = 7.0 * c4c_ * rcuti2_;
256 <      c6c_ = 9.0 * c5c_ * rcuti2_;
251 >      a2 = 0.0;
252 >      b0c = 1.0 / r;
253 >      b1c = (      b0c) / r2;
254 >      b2c = (3.0 * b1c) / r2;
255 >      b3c = (5.0 * b2c) / r2;
256 >      b4c = (7.0 * b3c) / r2;
257 >      b5c = (9.0 * b4c) / r2;
258 >      selfMult_ = b0c;
259      }
260 <  
261 <    if (summationMethod_ == esm_REACTION_FIELD) {
262 <      preRF_ = (dielectric_ - 1.0) /
263 <        ((2.0 * dielectric_ + 1.0) * cutoffRadius2_ * cutoffRadius_);
264 <      preRF2_ = 2.0 * preRF_;
265 <    }
260 >
261 >    // higher derivatives of B_0 at the cutoff radius:
262 >    db0c_1 = -r * b1c;
263 >    db0c_2 =     -b1c + r2 * b2c;
264 >    db0c_3 =          3.0*r*b2c  - r2*r*b3c;
265 >    db0c_4 =          3.0*b2c  - 6.0*r2*b3c     + r2*r2*b4c;
266 >    db0c_5 =                    -15.0*r*b3c + 10.0*r2*r*b4c - r2*r2*r*b5c;
267      
268 +    // working variables for the splines:
269 +    RealType ri, ri2;
270 +    RealType b0, b1, b2, b3, b4, b5;
271 +    RealType db0_1, db0_2, db0_3, db0_4, db0_5;
272 +    RealType f0;
273 +    RealType g0, g1, g2, g3, g4;
274 +    RealType h1, h2, h3, h4;
275 +    RealType s2, s3, s4;
276 +    RealType t3, t4;
277 +    RealType u4;
278 +
279 +    // working variables for Taylor expansion:
280 +    RealType rmRc, rmRc2, rmRc3, rmRc4;
281 +
282      // Add a 2 angstrom safety window to deal with cutoffGroups that
283      // have charged atoms longer than the cutoffRadius away from each
284 <    // other.  Splining may not be the best choice here.  Direct calls
285 <    // to erfc might be preferrable.
284 >    // other.  Splining is almost certainly the best choice here.
285 >    // Direct calls to erfc would be preferrable if it is a very fast
286 >    // implementation.
287  
288 <    RealType dx = (cutoffRadius_ + 2.0) / RealType(np_ - 1);
289 <    RealType rval;
290 <    vector<RealType> rvals;
291 <    vector<RealType> yvals;
292 <    for (int i = 0; i < np_; i++) {
293 <      rval = RealType(i) * dx;
294 <      rvals.push_back(rval);
295 <      yvals.push_back(erfc(dampingAlpha_ * rval));
288 >    RealType dx = (cutoffRadius_ + 2.0) / RealType(np_);
289 >
290 >    // Storage vectors for the computed functions    
291 >    vector<RealType> rv;
292 >    vector<RealType> v01v, v02v;
293 >    vector<RealType> v11v, v12v, v13v;
294 >    vector<RealType> v21v, v22v, v23v, v24v;
295 >    vector<RealType> v31v, v32v, v33v, v34v, v35v;
296 >    vector<RealType> v41v, v42v, v43v, v44v, v45v, v46v;
297 >
298 >    for (int i = 1; i < np_ + 1; i++) {
299 >      r = RealType(i) * dx;
300 >      rv.push_back(r);
301 >
302 >      ri = 1.0 / r;
303 >      ri2 = ri * ri;
304 >
305 >      r2 = r * r;
306 >      expTerm = exp(-a2 * r2);
307 >
308 >      // Taylor expansion factors (no need for factorials this way):
309 >      rmRc = r - cutoffRadius_;
310 >      rmRc2 = rmRc  * rmRc / 2.0;
311 >      rmRc3 = rmRc2 * rmRc / 3.0;
312 >      rmRc4 = rmRc3 * rmRc / 4.0;
313 >
314 >      // values of Smith's B_l functions at r:
315 >      if (screeningMethod_ == DAMPED) {            
316 >        b0 = erfc(dampingAlpha_ * r) * ri;
317 >        b1 = (      b0 +     2.0*a2     * expTerm * invArootPi) * ri2;
318 >        b2 = (3.0 * b1 + pow(2.0*a2, 2) * expTerm * invArootPi) * ri2;
319 >        b3 = (5.0 * b2 + pow(2.0*a2, 3) * expTerm * invArootPi) * ri2;
320 >        b4 = (7.0 * b3 + pow(2.0*a2, 4) * expTerm * invArootPi) * ri2;
321 >        b5 = (9.0 * b4 + pow(2.0*a2, 5) * expTerm * invArootPi) * ri2;
322 >      } else {
323 >        b0 = ri;
324 >        b1 = (      b0) * ri2;
325 >        b2 = (3.0 * b1) * ri2;
326 >        b3 = (5.0 * b2) * ri2;
327 >        b4 = (7.0 * b3) * ri2;
328 >        b5 = (9.0 * b4) * ri2;
329 >      }
330 >                
331 >      // higher derivatives of B_0 at r:
332 >      db0_1 = -r * b1;
333 >      db0_2 =     -b1 + r2 * b2;
334 >      db0_3 =          3.0*r*b2   - r2*r*b3;
335 >      db0_4 =          3.0*b2   - 6.0*r2*b3     + r2*r2*b4;
336 >      db0_5 =                    -15.0*r*b3 + 10.0*r2*r*b4 - r2*r2*r*b5;
337 >
338 >
339 >      switch (summationMethod_) {
340 >      case esm_SHIFTED_FORCE:
341 >        f0 = b0 - b0c - rmRc*db0c_1;
342 >        
343 >        g0 = db0_1 - db0c_1;
344 >        g1 = g0 - rmRc *db0c_2;
345 >        g2 = g1 - rmRc2*db0c_3;
346 >        g3 = g2 - rmRc3*db0c_4;
347 >        g4 = g3 - rmRc4*db0c_5;
348 >        
349 >        h1 = db0_2 - db0c_2;
350 >        h2 = h1 - rmRc *db0c_3;
351 >        h3 = h2 - rmRc2*db0c_4;
352 >        h4 = h3 - rmRc3*db0c_5;
353 >        
354 >        s2 = db0_3 - db0c_3;
355 >        s3 = s2 - rmRc *db0c_4;
356 >        s4 = s3 - rmRc2*db0c_5;
357 >        
358 >        t3 = db0_4 - db0c_4;
359 >        t4 = t3 - rmRc *db0c_5;
360 >        
361 >        u4 = db0_5 - db0c_5;
362 >        break;
363 >
364 >      case esm_SHIFTED_POTENTIAL:
365 >        f0 = b0 - b0c;
366 >        
367 >        g0 = db0_1;
368 >        g1 = db0_1 - db0c_1;
369 >        g2 = g1 - rmRc *db0c_2;
370 >        g3 = g2 - rmRc2*db0c_3;
371 >        g4 = g3 - rmRc3*db0c_4;
372 >
373 >        h1 = db0_2;
374 >        h2 = db0_2 - db0c_2;
375 >        h3 = h2 - rmRc *db0c_3;
376 >        h4 = h3 - rmRc2*db0c_4;
377 >        
378 >        s2 = db0_3;
379 >        s3 = db0_3 - db0c_3;
380 >        s4 = s3 - rmRc *db0c_4;
381 >
382 >        t3 = db0_4;
383 >        t4 = db0_4 - db0c_4;
384 >        
385 >        u4 = db0_5;
386 >        break;
387 >
388 >      case esm_SWITCHING_FUNCTION:
389 >      case esm_HARD:
390 >        f0 = b0;
391 >        
392 >        g0 = db0_1;
393 >        g1 = g0;
394 >        g2 = g1;
395 >        g3 = g2;
396 >        g4 = g3;
397 >        
398 >        h1 = db0_2;
399 >        h2 = h1;
400 >        h3 = h2;
401 >        h4 = h3;
402 >        
403 >        s2 = db0_3;
404 >        s3 = s2;
405 >        s4 = s3;
406 >        
407 >        t3 = db0_4;
408 >        t4 = t3;
409 >        
410 >        u4 = db0_5;
411 >        break;
412 >
413 >      case esm_REACTION_FIELD:
414 >
415 >        // following DL_POLY's lead for shifting the image charge potential:
416 >        f0 = b0  + preRF_ * r2
417 >          - (b0c + preRF_ * cutoffRadius_ * cutoffRadius_);
418 >
419 >        g0 = db0_1 + preRF_ * 2.0 * r;
420 >        g1 = g0;
421 >        g2 = g1;
422 >        g3 = g2;
423 >        g4 = g3;
424 >
425 >        h1 = db0_2 + preRF_ * 2.0;
426 >        h2 = h1;
427 >        h3 = h2;
428 >        h4 = h3;
429 >
430 >        s2 = db0_3;
431 >        s3 = s2;
432 >        s4 = s3;
433 >        
434 >        t3 = db0_4;
435 >        t4 = t3;
436 >        
437 >        u4 = db0_5;        
438 >        break;
439 >                
440 >      case esm_EWALD_FULL:
441 >      case esm_EWALD_PME:
442 >      case esm_EWALD_SPME:
443 >      default :
444 >        map<string, ElectrostaticSummationMethod>::iterator i;
445 >        std::string meth;
446 >        for (i = summationMap_.begin(); i != summationMap_.end(); ++i) {
447 >          if ((*i).second == summationMethod_) meth = (*i).first;
448 >        }
449 >        sprintf( painCave.errMsg,
450 >                 "Electrostatic::initialize: electrostaticSummationMethod %s \n"
451 >                 "\thas not been implemented yet. Please select one of:\n"
452 >                 "\t\"hard\", \"shifted_potential\", or \"shifted_force\"\n",
453 >                 meth.c_str() );
454 >        painCave.isFatal = 1;
455 >        simError();
456 >        break;      
457 >      }
458 >
459 >      v01 = f0;
460 >      v02 = g0;
461 >
462 >      v11 = g1;
463 >      v12 = g1 * ri;
464 >      v13 = h1 - v12;
465 >
466 >      v21 = g2 * ri;
467 >      v22 = h2 - v21;
468 >      v23 = v22 * ri;
469 >      v24 = s2 - 3.0*v23;        
470 >
471 >      v31 = (h3 - g3 * ri) * ri;
472 >      v32 = s3 - 3.0*v31;
473 >      v33 = v31 * ri;
474 >      v34 = v32 * ri;
475 >      v35 = t3 - 6.0*v34 - 3.0*v33;
476 >
477 >      v41 = (h4 - g4 * ri) * ri2;
478 >      v42 = s4 * ri - 3.0*v41;
479 >      v43 = t4 - 6.0*v42 - 3.0*v41;
480 >      v44 = v42 * ri;
481 >      v45 = v43 * ri;
482 >      v46 = u4 - 10.0*v45 - 15.0*v44;
483 >
484 >      // Add these computed values to the storage vectors for spline creation:
485 >      v01v.push_back(v01);
486 >      v02v.push_back(v02);
487 >
488 >      v11v.push_back(v11);
489 >      v12v.push_back(v12);
490 >      v13v.push_back(v13);
491 >
492 >      v21v.push_back(v21);
493 >      v22v.push_back(v22);
494 >      v23v.push_back(v23);
495 >      v24v.push_back(v24);
496 >
497 >      v31v.push_back(v31);
498 >      v32v.push_back(v32);
499 >      v33v.push_back(v33);
500 >      v34v.push_back(v34);
501 >      v35v.push_back(v35);
502 >      
503 >      v41v.push_back(v41);
504 >      v42v.push_back(v42);
505 >      v43v.push_back(v43);
506 >      v44v.push_back(v44);
507 >      v45v.push_back(v45);
508 >      v46v.push_back(v46);
509      }
269    erfcSpline_ = new CubicSpline();
270    erfcSpline_->addPoints(rvals, yvals);
271    haveElectroSpline_ = true;
510  
511 +    // construct the spline structures and fill them with the values we've
512 +    // computed:
513 +
514 +    v01s = new CubicSpline();
515 +    v01s->addPoints(rv, v01v);
516 +    v02s = new CubicSpline();
517 +    v02s->addPoints(rv, v02v);
518 +
519 +    v11s = new CubicSpline();
520 +    v11s->addPoints(rv, v11v);
521 +    v12s = new CubicSpline();
522 +    v12s->addPoints(rv, v12v);
523 +    v13s = new CubicSpline();
524 +    v13s->addPoints(rv, v13v);
525 +
526 +    v21s = new CubicSpline();
527 +    v21s->addPoints(rv, v21v);
528 +    v22s = new CubicSpline();
529 +    v22s->addPoints(rv, v22v);
530 +    v23s = new CubicSpline();
531 +    v23s->addPoints(rv, v23v);
532 +    v24s = new CubicSpline();
533 +    v24s->addPoints(rv, v24v);
534 +
535 +    v31s = new CubicSpline();
536 +    v31s->addPoints(rv, v31v);
537 +    v32s = new CubicSpline();
538 +    v32s->addPoints(rv, v32v);
539 +    v33s = new CubicSpline();
540 +    v33s->addPoints(rv, v33v);
541 +    v34s = new CubicSpline();
542 +    v34s->addPoints(rv, v34v);
543 +    v35s = new CubicSpline();
544 +    v35s->addPoints(rv, v35v);
545 +
546 +    v41s = new CubicSpline();
547 +    v41s->addPoints(rv, v41v);
548 +    v42s = new CubicSpline();
549 +    v42s->addPoints(rv, v42v);
550 +    v43s = new CubicSpline();
551 +    v43s->addPoints(rv, v43v);
552 +    v44s = new CubicSpline();
553 +    v44s->addPoints(rv, v44v);
554 +    v45s = new CubicSpline();
555 +    v45s->addPoints(rv, v45v);
556 +    v46s = new CubicSpline();
557 +    v46s->addPoints(rv, v46v);
558 +
559 +    haveElectroSplines_ = true;
560 +
561      initialized_ = true;
562    }
563        
# Line 278 | Line 566 | namespace OpenMD {
566      ElectrostaticAtomData electrostaticAtomData;
567      electrostaticAtomData.is_Charge = false;
568      electrostaticAtomData.is_Dipole = false;
281    electrostaticAtomData.is_SplitDipole = false;
569      electrostaticAtomData.is_Quadrupole = false;
570 +    electrostaticAtomData.is_Fluctuating = false;
571  
572      FixedChargeAdapter fca = FixedChargeAdapter(atomType);
573  
574      if (fca.isFixedCharge()) {
575        electrostaticAtomData.is_Charge = true;
576 <      electrostaticAtomData.charge = fca.getCharge();
576 >      electrostaticAtomData.fixedCharge = fca.getCharge();
577      }
578  
579      MultipoleAdapter ma = MultipoleAdapter(atomType);
580      if (ma.isMultipole()) {
581        if (ma.isDipole()) {
582          electrostaticAtomData.is_Dipole = true;
583 <        electrostaticAtomData.dipole_moment = ma.getDipoleMoment();
583 >        electrostaticAtomData.dipole = ma.getDipole();
584        }
297      if (ma.isSplitDipole()) {
298        electrostaticAtomData.is_SplitDipole = true;
299        electrostaticAtomData.split_dipole_distance = ma.getSplitDipoleDistance();
300      }
585        if (ma.isQuadrupole()) {
302        // Quadrupoles in OpenMD are set as the diagonal elements
303        // of the diagonalized traceless quadrupole moment tensor.
304        // The column vectors of the unitary matrix that diagonalizes
305        // the quadrupole moment tensor become the eFrame (or the
306        // electrostatic version of the body-fixed frame.
586          electrostaticAtomData.is_Quadrupole = true;
587 <        electrostaticAtomData.quadrupole_moments = ma.getQuadrupoleMoments();
587 >        electrostaticAtomData.quadrupole = ma.getQuadrupole();
588        }
589      }
590      
591 +    FluctuatingChargeAdapter fqa = FluctuatingChargeAdapter(atomType);
592  
593 +    if (fqa.isFluctuatingCharge()) {
594 +      electrostaticAtomData.is_Fluctuating = true;
595 +      electrostaticAtomData.electronegativity = fqa.getElectronegativity();
596 +      electrostaticAtomData.hardness = fqa.getHardness();
597 +      electrostaticAtomData.slaterN = fqa.getSlaterN();
598 +      electrostaticAtomData.slaterZeta = fqa.getSlaterZeta();
599 +    }
600 +
601      pair<map<int,AtomType*>::iterator,bool> ret;    
602      ret = ElectrostaticList.insert( pair<int,AtomType*>(atomType->getIdent(),
603                                                          atomType) );
# Line 322 | Line 610 | namespace OpenMD {
610        simError();        
611      }
612      
613 <    ElectrostaticMap[atomType] = electrostaticAtomData;    
613 >    ElectrostaticMap[atomType] = electrostaticAtomData;  
614 >
615 >    // Now, iterate over all known types and add to the mixing map:
616 >    
617 >    map<AtomType*, ElectrostaticAtomData>::iterator it;
618 >    for( it = ElectrostaticMap.begin(); it != ElectrostaticMap.end(); ++it) {
619 >      AtomType* atype2 = (*it).first;
620 >      ElectrostaticAtomData eaData2 = (*it).second;
621 >      if (eaData2.is_Fluctuating && electrostaticAtomData.is_Fluctuating) {
622 >        
623 >        RealType a = electrostaticAtomData.slaterZeta;
624 >        RealType b = eaData2.slaterZeta;
625 >        int m = electrostaticAtomData.slaterN;
626 >        int n = eaData2.slaterN;
627 >
628 >        // Create the spline of the coulombic integral for s-type
629 >        // Slater orbitals.  Add a 2 angstrom safety window to deal
630 >        // with cutoffGroups that have charged atoms longer than the
631 >        // cutoffRadius away from each other.
632 >
633 >        RealType rval;
634 >        RealType dr = (cutoffRadius_ + 2.0) / RealType(np_ - 1);
635 >        vector<RealType> rvals;
636 >        vector<RealType> Jvals;
637 >        // don't start at i = 0, as rval = 0 is undefined for the
638 >        // slater overlap integrals.
639 >        for (int i = 1; i < np_+1; i++) {
640 >          rval = RealType(i) * dr;
641 >          rvals.push_back(rval);
642 >          Jvals.push_back(sSTOCoulInt( a, b, m, n, rval *
643 >                                       PhysicalConstants::angstromToBohr ) *
644 >                          PhysicalConstants::hartreeToKcal );
645 >        }
646 >        
647 >        CubicSpline* J = new CubicSpline();
648 >        J->addPoints(rvals, Jvals);
649 >        
650 >        pair<AtomType*, AtomType*> key1, key2;
651 >        key1 = make_pair(atomType, atype2);
652 >        key2 = make_pair(atype2, atomType);
653 >        
654 >        Jij[key1] = J;
655 >        Jij[key2] = J;
656 >      }
657 >    }
658 >
659      return;
660    }
661    
662    void Electrostatic::setCutoffRadius( RealType rCut ) {
663      cutoffRadius_ = rCut;
331    rrf_ = cutoffRadius_;
664      haveCutoffRadius_ = true;
665    }
666  
335  void Electrostatic::setSwitchingRadius( RealType rSwitch ) {
336    rt_ = rSwitch;
337  }
667    void Electrostatic::setElectrostaticSummationMethod( ElectrostaticSummationMethod esm ) {
668      summationMethod_ = esm;
669    }
# Line 352 | Line 681 | namespace OpenMD {
681  
682    void Electrostatic::calcForce(InteractionData &idat) {
683  
684 <    // utility variables.  Should clean these up and use the Vector3d and
685 <    // Mat3x3d to replace as many as we can in future versions:
684 >    RealType C_a, C_b;  // Charges
685 >    Vector3d D_a, D_b;  // Dipoles (space-fixed)
686 >    Mat3x3d  Q_a, Q_b;  // Quadrupoles (space-fixed)
687  
688 <    RealType q_i, q_j, mu_i, mu_j, d_i, d_j;
689 <    RealType qxx_i, qyy_i, qzz_i;
690 <    RealType qxx_j, qyy_j, qzz_j;
691 <    RealType cx_i, cy_i, cz_i;
692 <    RealType cx_j, cy_j, cz_j;
693 <    RealType cx2, cy2, cz2;
364 <    RealType ct_i, ct_j, ct_ij, a1;
365 <    RealType riji, ri, ri2, ri3, ri4;
366 <    RealType pref, vterm, epot, dudr;
367 <    RealType vpair(0.0);
368 <    RealType scale, sc2;
369 <    RealType pot_term, preVal, rfVal;
370 <    RealType c2ri, c3ri, c4rij, cti3, ctj3, ctidotj;
371 <    RealType preSw, preSwSc;
372 <    RealType c1, c2, c3, c4;
373 <    RealType erfcVal(1.0), derfcVal(0.0);
374 <    RealType BigR;
375 <    RealType two(2.0), three(3.0);
688 >    RealType ri, ri2, ri3, ri4;                  // Distance utility scalars
689 >    RealType rdDa, rdDb;                         // Dipole utility scalars
690 >    Vector3d rxDa, rxDb;                         // Dipole utility vectors
691 >    RealType rdQar, rdQbr, trQa, trQb;           // Quadrupole utility scalars
692 >    Vector3d Qar, Qbr, rQa, rQb, rxQar, rxQbr;   // Quadrupole utility vectors
693 >    RealType pref;
694  
695 <    Vector3d Q_i, Q_j;
696 <    Vector3d ux_i, uy_i, uz_i;
697 <    Vector3d ux_j, uy_j, uz_j;
698 <    Vector3d dudux_i, duduy_i, duduz_i;
699 <    Vector3d dudux_j, duduy_j, duduz_j;
382 <    Vector3d rhatdot2, rhatc4;
383 <    Vector3d dVdr;
695 >    RealType DadDb, trQaQb, DadQbr, DbdQar;       // Cross-interaction scalars
696 >    RealType rQaQbr;
697 >    Vector3d DaxDb, DadQb, DbdQa, DaxQbr, DbxQar; // Cross-interaction vectors
698 >    Vector3d rQaQb, QaQbr, QaxQb, rQaxQbr;
699 >    Mat3x3d  QaQb;                                // Cross-interaction matrices
700  
701 <    // variables for indirect (reaction field) interactions for excluded pairs:
702 <    RealType indirect_Pot(0.0);
703 <    RealType indirect_vpair(0.0);
704 <    Vector3d indirect_dVdr(V3Zero);
705 <    Vector3d indirect_duduz_i(V3Zero), indirect_duduz_j(V3Zero);
701 >    RealType U(0.0);  // Potential
702 >    Vector3d F(0.0);  // Force
703 >    Vector3d Ta(0.0); // Torque on site a
704 >    Vector3d Tb(0.0); // Torque on site b
705 >    Vector3d Ea(0.0); // Electric field at site a
706 >    Vector3d Eb(0.0); // Electric field at site b
707 >    RealType dUdCa(0.0); // fluctuating charge force at site a
708 >    RealType dUdCb(0.0); // fluctuating charge force at site a
709 >    
710 >    // Indirect interactions mediated by the reaction field.
711 >    RealType indirect_Pot(0.0);  // Potential
712 >    Vector3d indirect_F(0.0);    // Force
713 >    Vector3d indirect_Ta(0.0);   // Torque on site a
714 >    Vector3d indirect_Tb(0.0);   // Torque on site b
715  
716 <    pair<RealType, RealType> res;
716 >    // Excluded potential that is still computed for fluctuating charges
717 >    RealType excluded_Pot(0.0);
718 >
719 >    RealType rfContrib, coulInt;
720      
721 +    // spline for coulomb integral
722 +    CubicSpline* J;
723 +
724      if (!initialized_) initialize();
725      
726      ElectrostaticAtomData data1 = ElectrostaticMap[idat.atypes.first];
# Line 397 | Line 728 | namespace OpenMD {
728      
729      // some variables we'll need independent of electrostatic type:
730  
731 <    riji = 1.0 /  *(idat.rij) ;
732 <    Vector3d rhat =  *(idat.d)   * riji;
733 <
731 >    ri = 1.0 /  *(idat.rij);
732 >    Vector3d rhat =  *(idat.d)  * ri;
733 >    ri2 = ri * ri;
734 >      
735      // logicals
736  
737 <    bool i_is_Charge = data1.is_Charge;
738 <    bool i_is_Dipole = data1.is_Dipole;
739 <    bool i_is_SplitDipole = data1.is_SplitDipole;
740 <    bool i_is_Quadrupole = data1.is_Quadrupole;
737 >    bool a_is_Charge = data1.is_Charge;
738 >    bool a_is_Dipole = data1.is_Dipole;
739 >    bool a_is_Quadrupole = data1.is_Quadrupole;
740 >    bool a_is_Fluctuating = data1.is_Fluctuating;
741  
742 <    bool j_is_Charge = data2.is_Charge;
743 <    bool j_is_Dipole = data2.is_Dipole;
744 <    bool j_is_SplitDipole = data2.is_SplitDipole;
745 <    bool j_is_Quadrupole = data2.is_Quadrupole;
742 >    bool b_is_Charge = data2.is_Charge;
743 >    bool b_is_Dipole = data2.is_Dipole;
744 >    bool b_is_Quadrupole = data2.is_Quadrupole;
745 >    bool b_is_Fluctuating = data2.is_Fluctuating;
746 >
747 >    // Obtain all of the required radial function values from the
748 >    // spline structures:
749      
750 <    if (i_is_Charge) {
751 <      q_i = data1.charge;
752 <      if (idat.excluded) {
418 <        *(idat.skippedCharge2) += q_i;
419 <      }
750 >    // needed for fields (and forces):
751 >    if (a_is_Charge || b_is_Charge) {
752 >      v02 = v02s->getValueAt( *(idat.rij) );
753      }
754 <
755 <    if (i_is_Dipole) {
756 <      mu_i = data1.dipole_moment;
424 <      uz_i = idat.eFrame1->getColumn(2);
425 <      
426 <      ct_i = dot(uz_i, rhat);
427 <
428 <      if (i_is_SplitDipole)
429 <        d_i = data1.split_dipole_distance;
430 <      
431 <      duduz_i = V3Zero;
754 >    if (a_is_Dipole || b_is_Dipole) {
755 >      v12 = v12s->getValueAt( *(idat.rij) );
756 >      v13 = v13s->getValueAt( *(idat.rij) );
757      }
758 <    
759 <    if (i_is_Quadrupole) {
760 <      Q_i = data1.quadrupole_moments;
436 <      qxx_i = Q_i.x();
437 <      qyy_i = Q_i.y();
438 <      qzz_i = Q_i.z();
439 <      
440 <      ux_i = idat.eFrame1->getColumn(0);
441 <      uy_i = idat.eFrame1->getColumn(1);
442 <      uz_i = idat.eFrame1->getColumn(2);
443 <
444 <      cx_i = dot(ux_i, rhat);
445 <      cy_i = dot(uy_i, rhat);
446 <      cz_i = dot(uz_i, rhat);
447 <
448 <      dudux_i = V3Zero;
449 <      duduy_i = V3Zero;
450 <      duduz_i = V3Zero;
758 >    if (a_is_Quadrupole || b_is_Quadrupole) {
759 >      v23 = v23s->getValueAt( *(idat.rij) );
760 >      v24 = v24s->getValueAt( *(idat.rij) );
761      }
762  
763 <    if (j_is_Charge) {
764 <      q_j = data2.charge;
765 <      if (idat.excluded) {
456 <        *(idat.skippedCharge1) += q_j;
457 <      }
763 >    // needed for potentials (and torques):
764 >    if (a_is_Charge && b_is_Charge) {
765 >      v01 = v01s->getValueAt( *(idat.rij) );
766      }
767 +    if ((a_is_Charge && b_is_Dipole) || (b_is_Charge && a_is_Dipole)) {
768 +      v11 = v11s->getValueAt( *(idat.rij) );
769 +    }
770 +    if ((a_is_Charge && b_is_Quadrupole) || (b_is_Charge && a_is_Quadrupole)) {
771 +      v21 = v21s->getValueAt( *(idat.rij) );
772 +      v22 = v22s->getValueAt( *(idat.rij) );
773 +    } else if (a_is_Dipole && b_is_Dipole) {
774 +      v21 = v21s->getValueAt( *(idat.rij) );
775 +      v22 = v22s->getValueAt( *(idat.rij) );
776 +      v23 = v23s->getValueAt( *(idat.rij) );
777 +      v24 = v24s->getValueAt( *(idat.rij) );
778 +    }
779 +    if ((a_is_Dipole && b_is_Quadrupole) ||
780 +        (b_is_Dipole && a_is_Quadrupole)) {
781 +      v31 = v31s->getValueAt( *(idat.rij) );
782 +      v32 = v32s->getValueAt( *(idat.rij) );
783 +      v33 = v33s->getValueAt( *(idat.rij) );
784 +      v34 = v34s->getValueAt( *(idat.rij) );
785 +      v35 = v35s->getValueAt( *(idat.rij) );
786 +    }
787 +    if (a_is_Quadrupole && b_is_Quadrupole) {
788 +      v41 = v41s->getValueAt( *(idat.rij) );
789 +      v42 = v42s->getValueAt( *(idat.rij) );
790 +      v43 = v43s->getValueAt( *(idat.rij) );
791 +      v44 = v44s->getValueAt( *(idat.rij) );
792 +      v45 = v45s->getValueAt( *(idat.rij) );
793 +      v46 = v46s->getValueAt( *(idat.rij) );
794 +    }
795  
796 <
797 <    if (j_is_Dipole) {
798 <      mu_j = data2.dipole_moment;
799 <      uz_j = idat.eFrame2->getColumn(2);
796 >    // calculate the single-site contributions (fields, etc).
797 >    
798 >    if (a_is_Charge) {
799 >      C_a = data1.fixedCharge;
800        
801 <      ct_j = dot(uz_j, rhat);
802 <
803 <      if (j_is_SplitDipole)
468 <        d_j = data2.split_dipole_distance;
801 >      if (a_is_Fluctuating) {
802 >        C_a += *(idat.flucQ1);
803 >      }
804        
805 <      duduz_j = V3Zero;
805 >      if (idat.excluded) {
806 >        *(idat.skippedCharge2) += C_a;
807 >      }
808 >      Eb -= C_a *  pre11_ * v02 * rhat;
809      }
810      
811 <    if (j_is_Quadrupole) {
812 <      Q_j = data2.quadrupole_moments;
813 <      qxx_j = Q_j.x();
814 <      qyy_j = Q_j.y();
815 <      qzz_j = Q_j.z();
478 <      
479 <      ux_j = idat.eFrame2->getColumn(0);
480 <      uy_j = idat.eFrame2->getColumn(1);
481 <      uz_j = idat.eFrame2->getColumn(2);
482 <
483 <      cx_j = dot(ux_j, rhat);
484 <      cy_j = dot(uy_j, rhat);
485 <      cz_j = dot(uz_j, rhat);
486 <
487 <      dudux_j = V3Zero;
488 <      duduy_j = V3Zero;
489 <      duduz_j = V3Zero;
811 >    if (a_is_Dipole) {
812 >      D_a = *(idat.dipole1);
813 >      rdDa = dot(rhat, D_a);
814 >      rxDa = cross(rhat, D_a);
815 >      Eb -=  pre12_ * (v13 * rdDa * rhat + v12 * D_a);
816      }
817      
818 <    epot = 0.0;
819 <    dVdr = V3Zero;
818 >    if (a_is_Quadrupole) {
819 >      Q_a = *(idat.quadrupole1);
820 >      trQa =  Q_a.trace();
821 >      Qar =   Q_a * rhat;
822 >      rQa = rhat * Q_a;
823 >      rdQar = dot(rhat, Qar);
824 >      rxQar = cross(rhat, Qar);
825 >      Eb -= pre14_ * ((trQa * rhat + 2.0 * Qar) * v23 + rdQar * rhat * v24);
826 >    }
827      
828 <    if (i_is_Charge) {
829 <      
830 <      if (j_is_Charge) {
831 <        if (screeningMethod_ == DAMPED) {
832 <          // assemble the damping variables
833 <          //res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
834 <          //erfcVal = res.first;
835 <          //derfcVal = res.second;
836 <
837 <          erfcVal = erfc(dampingAlpha_ * *(idat.rij));
838 <          derfcVal = - alphaPi_ * exp(-alpha2_ * *(idat.r2));
828 >    if (b_is_Charge) {
829 >      C_b = data2.fixedCharge;
830 >      
831 >      if (b_is_Fluctuating)
832 >        C_b += *(idat.flucQ2);
833 >      
834 >      if (idat.excluded) {
835 >        *(idat.skippedCharge1) += C_b;
836 >      }
837 >      Ea += C_b *  pre11_ * v02 * rhat;
838 >    }
839 >    
840 >    if (b_is_Dipole) {
841 >      D_b = *(idat.dipole2);
842 >      rdDb = dot(rhat, D_b);
843 >      rxDb = cross(rhat, D_b);
844 >      Ea += pre12_ * (v13 * rdDb * rhat + v12 * D_b);
845 >    }
846 >    
847 >    if (b_is_Quadrupole) {
848 >      Q_b = *(idat.quadrupole2);
849 >      trQb =  Q_b.trace();
850 >      Qbr =   Q_b * rhat;
851 >      rQb = rhat * Q_b;
852 >      rdQbr = dot(rhat, Qbr);
853 >      rxQbr = cross(rhat, Qbr);
854 >      Ea += pre14_ * ((trQb * rhat + 2.0 * Qbr) * v23 + rdQbr * rhat * v24);
855 >    }
856 >    
857 >    if ((a_is_Fluctuating || b_is_Fluctuating) && idat.excluded) {
858 >      J = Jij[idat.atypes];
859 >    }    
860 >    
861 >    if (a_is_Charge) {    
862 >      
863 >      if (b_is_Charge) {
864 >        pref =  pre11_ * *(idat.electroMult);          
865 >        U  += C_a * C_b * pref * v01;
866 >        F  += C_a * C_b * pref * v02 * rhat;
867 >        
868 >        // If this is an excluded pair, there are still indirect
869 >        // interactions via the reaction field we must worry about:
870  
871 <          c1 = erfcVal * riji;
872 <          c2 = (-derfcVal + c1) * riji;
873 <        } else {
874 <          c1 = riji;
511 <          c2 = c1 * riji;
871 >        if (summationMethod_ == esm_REACTION_FIELD && idat.excluded) {
872 >          rfContrib = preRF_ * pref * C_a * C_b * *(idat.r2);
873 >          indirect_Pot += rfContrib;
874 >          indirect_F   += rfContrib * 2.0 * ri * rhat;
875          }
513
514        preVal =  *(idat.electroMult) * pre11_ * q_i * q_j;
876          
877 <        if (summationMethod_ == esm_SHIFTED_POTENTIAL) {
878 <          vterm = preVal * (c1 - c1c_);
879 <          dudr  = - *(idat.sw)  * preVal * c2;
877 >        // Fluctuating charge forces are handled via Coulomb integrals
878 >        // for excluded pairs (i.e. those connected via bonds) and
879 >        // with the standard charge-charge interaction otherwise.
880  
881 <        } else if (summationMethod_ == esm_SHIFTED_FORCE)  {
882 <          vterm = preVal * ( c1 - c1c_ + c2c_*( *(idat.rij)  - cutoffRadius_) );
883 <          dudr  =  *(idat.sw)  * preVal * (c2c_ - c2);
881 >        if (idat.excluded) {          
882 >          if (a_is_Fluctuating || b_is_Fluctuating) {
883 >            coulInt = J->getValueAt( *(idat.rij) );
884 >            if (a_is_Fluctuating)  dUdCa += coulInt * C_b;
885 >            if (b_is_Fluctuating)  dUdCb += coulInt * C_a;
886 >            excluded_Pot += C_a * C_b * coulInt;
887 >          }          
888 >        } else {
889 >          if (a_is_Fluctuating) dUdCa += C_b * pref * v01;
890 >          if (a_is_Fluctuating) dUdCb += C_a * pref * v01;
891 >        }
892 >      }
893  
894 <        } else if (summationMethod_ == esm_REACTION_FIELD) {
895 <          rfVal = preRF_ *  *(idat.rij)  *  *(idat.rij);
894 >      if (b_is_Dipole) {
895 >        pref =  pre12_ * *(idat.electroMult);        
896 >        U  += C_a * pref * v11 * rdDb;
897 >        F  += C_a * pref * (v13 * rdDb * rhat + v12 * D_b);
898 >        Tb += C_a * pref * v11 * rxDb;
899  
900 <          vterm = preVal * ( riji + rfVal );            
528 <          dudr  =  *(idat.sw)  * preVal * ( 2.0 * rfVal - riji ) * riji;
529 <          
530 <          // if this is an excluded pair, there are still indirect
531 <          // interactions via the reaction field we must worry about:
900 >        if (a_is_Fluctuating) dUdCa += pref * v11 * rdDb;
901  
902 <          if (idat.excluded) {
903 <            indirect_vpair += preVal * rfVal;
904 <            indirect_Pot += *(idat.sw) * preVal * rfVal;
536 <            indirect_dVdr += *(idat.sw)  * preVal * two * rfVal  * riji * rhat;
537 <          }
538 <          
539 <        } else {
902 >        // Even if we excluded this pair from direct interactions, we
903 >        // still have the reaction-field-mediated charge-dipole
904 >        // interaction:
905  
906 <          vterm = preVal * riji * erfcVal;          
907 <          dudr  = -  *(idat.sw)  * preVal * c2;
908 <
906 >        if (summationMethod_ == esm_REACTION_FIELD && idat.excluded) {
907 >          rfContrib = C_a * pref * preRF_ * 2.0 * *(idat.rij);
908 >          indirect_Pot += rfContrib * rdDb;
909 >          indirect_F   += rfContrib * D_b / (*idat.rij);
910 >          indirect_Tb  += C_a * pref * preRF_ * rxDb;
911          }
545
546        vpair += vterm;
547        epot +=  *(idat.sw)  * vterm;
548        dVdr += dudr * rhat;                
912        }
913  
914 <      if (j_is_Dipole) {
915 <        // pref is used by all the possible methods
916 <        pref =  *(idat.electroMult) * pre12_ * q_i * mu_j;
917 <        preSw =  *(idat.sw)  * pref;
914 >      if (b_is_Quadrupole) {
915 >        pref = pre14_ * *(idat.electroMult);
916 >        U  +=  C_a * pref * (v21 * trQb + v22 * rdQbr);
917 >        F  +=  C_a * pref * (trQb * rhat + 2.0 * Qbr) * v23;
918 >        F  +=  C_a * pref * rdQbr * rhat * v24;
919 >        Tb +=  C_a * pref * 2.0 * rxQbr * v22;
920  
921 <        if (summationMethod_ == esm_REACTION_FIELD) {
922 <          ri2 = riji * riji;
923 <          ri3 = ri2 * riji;
559 <    
560 <          vterm = - pref * ct_j * ( ri2 - preRF2_ *  *(idat.rij)  );
561 <          vpair += vterm;
562 <          epot +=  *(idat.sw)  * vterm;
921 >        if (a_is_Fluctuating) dUdCa += pref * (v21 * trQb + v22 * rdQbr);
922 >      }
923 >    }
924  
925 <          dVdr +=  -preSw * (ri3 * (uz_j - three * ct_j * rhat) - preRF2_*uz_j);
565 <          duduz_j += -preSw * rhat * (ri2 - preRF2_ *  *(idat.rij) );  
925 >    if (a_is_Dipole) {
926  
927 <          // Even if we excluded this pair from direct interactions,
928 <          // we still have the reaction-field-mediated charge-dipole
569 <          // interaction:
927 >      if (b_is_Charge) {
928 >        pref = pre12_ * *(idat.electroMult);
929  
930 <          if (idat.excluded) {
931 <            indirect_vpair += pref * ct_j * preRF2_ * *(idat.rij);
932 <            indirect_Pot += preSw * ct_j * preRF2_ * *(idat.rij);
574 <            indirect_dVdr += preSw * preRF2_ * uz_j;
575 <            indirect_duduz_j += preSw * rhat * preRF2_ *  *(idat.rij);
576 <          }
577 <                      
578 <        } else {
579 <          // determine the inverse r used if we have split dipoles
580 <          if (j_is_SplitDipole) {
581 <            BigR = sqrt( *(idat.r2) + 0.25 * d_j * d_j);
582 <            ri = 1.0 / BigR;
583 <            scale =  *(idat.rij)  * ri;
584 <          } else {
585 <            ri = riji;
586 <            scale = 1.0;
587 <          }
588 <          
589 <          sc2 = scale * scale;
930 >        U  -= C_b * pref * v11 * rdDa;
931 >        F  -= C_b * pref * (v13 * rdDa * rhat + v12 * D_a);
932 >        Ta -= C_b * pref * v11 * rxDa;
933  
934 <          if (screeningMethod_ == DAMPED) {
592 <            // assemble the damping variables
593 <            //res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
594 <            //erfcVal = res.first;
595 <            //derfcVal = res.second;
596 <            erfcVal = erfc(dampingAlpha_ * *(idat.rij));
597 <            derfcVal = - alphaPi_ * exp(-alpha2_ * *(idat.r2));
598 <            c1 = erfcVal * ri;
599 <            c2 = (-derfcVal + c1) * ri;
600 <            c3 = -2.0 * derfcVal * alpha2_ + 3.0 * c2 * ri;
601 <          } else {
602 <            c1 = ri;
603 <            c2 = c1 * ri;
604 <            c3 = 3.0 * c2 * ri;
605 <          }
606 <            
607 <          c2ri = c2 * ri;
934 >        if (b_is_Fluctuating) dUdCb -= pref * v11 * rdDa;
935  
936 <          // calculate the potential
937 <          pot_term =  scale * c2;
938 <          vterm = -pref * ct_j * pot_term;
939 <          vpair += vterm;
940 <          epot +=  *(idat.sw)  * vterm;
941 <            
942 <          // calculate derivatives for forces and torques
943 <
617 <          dVdr += -preSw * (uz_j * c2ri - ct_j * rhat * sc2 * c3);
618 <          duduz_j += -preSw * pot_term * rhat;
619 <
936 >        // Even if we excluded this pair from direct interactions,
937 >        // we still have the reaction-field-mediated charge-dipole
938 >        // interaction:
939 >        if (summationMethod_ == esm_REACTION_FIELD && idat.excluded) {
940 >          rfContrib = C_b * pref * preRF_ * 2.0 * *(idat.rij);
941 >          indirect_Pot -= rfContrib * rdDa;
942 >          indirect_F   -= rfContrib * D_a / (*idat.rij);
943 >          indirect_Ta  -= C_b * pref * preRF_ * rxDa;
944          }
945        }
946  
947 <      if (j_is_Quadrupole) {
948 <        // first precalculate some necessary variables
949 <        cx2 = cx_j * cx_j;
950 <        cy2 = cy_j * cy_j;
951 <        cz2 = cz_j * cz_j;
952 <        pref =   *(idat.electroMult) * pre14_ * q_i * one_third_;
953 <          
954 <        if (screeningMethod_ == DAMPED) {
955 <          // assemble the damping variables
956 <          //res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
957 <          //erfcVal = res.first;
958 <          //derfcVal = res.second;
959 <          erfcVal = erfc(dampingAlpha_ * *(idat.rij));
960 <          derfcVal = - alphaPi_ * exp(-alpha2_ * *(idat.r2));
961 <          c1 = erfcVal * riji;
962 <          c2 = (-derfcVal + c1) * riji;
963 <          c3 = -2.0 * derfcVal * alpha2_ + 3.0 * c2 * riji;
964 <          c4 = -4.0 * derfcVal * alpha4_ + 5.0 * c3 * riji * riji;
965 <        } else {
642 <          c1 = riji;
643 <          c2 = c1 * riji;
644 <          c3 = 3.0 * c2 * riji;
645 <          c4 = 5.0 * c3 * riji * riji;
947 >      if (b_is_Dipole) {
948 >        pref = pre22_ * *(idat.electroMult);
949 >        DadDb = dot(D_a, D_b);
950 >        DaxDb = cross(D_a, D_b);
951 >
952 >        U  -= pref * (DadDb * v21 + rdDa * rdDb * v22);
953 >        F  -= pref * (DadDb * rhat + rdDb * D_a + rdDa * D_b)*v23;
954 >        F  -= pref * (rdDa * rdDb) * v24 * rhat;
955 >        Ta += pref * ( v21 * DaxDb - v22 * rdDb * rxDa);
956 >        Tb += pref * (-v21 * DaxDb - v22 * rdDa * rxDb);
957 >
958 >        // Even if we excluded this pair from direct interactions, we
959 >        // still have the reaction-field-mediated dipole-dipole
960 >        // interaction:
961 >        if (summationMethod_ == esm_REACTION_FIELD && idat.excluded) {
962 >          rfContrib = -pref * preRF_ * 2.0;
963 >          indirect_Pot += rfContrib * DadDb;
964 >          indirect_Ta  += rfContrib * DaxDb;
965 >          indirect_Tb  -= rfContrib * DaxDb;
966          }
967  
968 <        // precompute variables for convenience
649 <        preSw =  *(idat.sw)  * pref;
650 <        c2ri = c2 * riji;
651 <        c3ri = c3 * riji;
652 <        c4rij = c4 *  *(idat.rij) ;
653 <        rhatdot2 = two * rhat * c3;
654 <        rhatc4 = rhat * c4rij;
968 >      }
969  
970 <        // calculate the potential
971 <        pot_term = ( qxx_j * (cx2*c3 - c2ri) +
972 <                     qyy_j * (cy2*c3 - c2ri) +
973 <                     qzz_j * (cz2*c3 - c2ri) );
974 <        vterm = pref * pot_term;
661 <        vpair += vterm;
662 <        epot +=  *(idat.sw)  * vterm;
663 <                
664 <        // calculate derivatives for the forces and torques
970 >      if (b_is_Quadrupole) {
971 >        pref = pre24_ * *(idat.electroMult);
972 >        DadQb = D_a * Q_b;
973 >        DadQbr = dot(D_a, Qbr);
974 >        DaxQbr = cross(D_a, Qbr);
975  
976 <        dVdr += -preSw * ( qxx_j* (cx2*rhatc4 - (two*cx_j*ux_j + rhat)*c3ri) +
977 <                           qyy_j* (cy2*rhatc4 - (two*cy_j*uy_j + rhat)*c3ri) +
978 <                           qzz_j* (cz2*rhatc4 - (two*cz_j*uz_j + rhat)*c3ri));
979 <                          
980 <        dudux_j += preSw * qxx_j * cx_j * rhatdot2;
981 <        duduy_j += preSw * qyy_j * cy_j * rhatdot2;
982 <        duduz_j += preSw * qzz_j * cz_j * rhatdot2;
976 >        U  -= pref * ((trQb*rdDa + 2.0*DadQbr)*v31 + rdDa*rdQbr*v32);
977 >        F  -= pref * (trQb*D_a + 2.0*DadQb) * v33;
978 >        F  -= pref * (trQb*rdDa*rhat + 2.0*DadQbr*rhat + D_a*rdQbr
979 >                      + 2.0*rdDa*rQb)*v34;
980 >        F  -= pref * (rdDa * rdQbr * rhat * v35);
981 >        Ta += pref * ((-trQb*rxDa + 2.0 * DaxQbr)*v31 - rxDa*rdQbr*v32);
982 >        Tb += pref * ((2.0*cross(DadQb, rhat) - 2.0*DaxQbr)*v31
983 >                      - 2.0*rdDa*rxQbr*v32);
984        }
985      }
675    
676    if (i_is_Dipole) {
986  
987 <      if (j_is_Charge) {
988 <        // variables used by all the methods
989 <        pref =  *(idat.electroMult) * pre12_ * q_j * mu_i;
990 <        preSw =  *(idat.sw)  * pref;
987 >    if (a_is_Quadrupole) {
988 >      if (b_is_Charge) {
989 >        pref = pre14_ * *(idat.electroMult);
990 >        U  += C_b * pref * (v21 * trQa + v22 * rdQar);
991 >        F  += C_b * pref * (trQa * rhat + 2.0 * Qar) * v23;
992 >        F  += C_b * pref * rdQar * rhat * v24;
993 >        Ta += C_b * pref * 2.0 * rxQar * v22;
994  
995 <        if (summationMethod_ == esm_REACTION_FIELD) {
995 >        if (b_is_Fluctuating) dUdCb += pref * (v21 * trQa + v22 * rdQar);
996 >      }
997 >      if (b_is_Dipole) {
998 >        pref = pre24_ * *(idat.electroMult);
999 >        DbdQa = D_b * Q_a;
1000 >        DbdQar = dot(D_b, Qar);
1001 >        DbxQar = cross(D_b, Qar);
1002  
1003 <          ri2 = riji * riji;
1004 <          ri3 = ri2 * riji;
1003 >        U  += pref * ((trQa*rdDb + 2.0*DbdQar)*v31 + rdDb*rdQar*v32);
1004 >        F  += pref * (trQa*D_b + 2.0*DbdQa) * v33;
1005 >        F  += pref * (trQa*rdDb*rhat + 2.0*DbdQar*rhat + D_b*rdQar
1006 >                      + 2.0*rdDb*rQa)*v34;
1007 >        F  += pref * (rdDb * rdQar * rhat * v35);
1008 >        Ta += pref * ((-2.0*cross(DbdQa, rhat) + 2.0*DbxQar)*v31
1009 >                      + 2.0*rdDb*rxQar*v32);
1010 >        Tb += pref * ((trQa*rxDb - 2.0 * DbxQar)*v31 + rxDb*rdQar*v32);
1011 >      }
1012 >      if (b_is_Quadrupole) {
1013 >        pref = pre44_ * *(idat.electroMult);  // yes
1014 >        QaQb = Q_a * Q_b;
1015 >        trQaQb = QaQb.trace();
1016 >        rQaQb = rhat * QaQb;
1017 >        QaQbr = QaQb * rhat;
1018 >        QaxQb = cross(Q_a, Q_b);
1019 >        rQaQbr = dot(rQa, Qbr);
1020 >        rQaxQbr = cross(rQa, Qbr);
1021 >        
1022 >        U  += pref * (trQa * trQb + 2.0 * trQaQb) * v41;
1023 >        U  += pref * (trQa * rdQbr + trQb * rdQar  + 4.0 * rQaQbr) * v42;
1024 >        U  += pref * (rdQar * rdQbr) * v43;
1025  
1026 <          vterm = pref * ct_i * ( ri2 - preRF2_ *  *(idat.rij)  );
1027 <          vpair += vterm;
1028 <          epot +=  *(idat.sw)  * vterm;
691 <          
692 <          dVdr += preSw * (ri3 * (uz_i - three * ct_i * rhat) - preRF2_ * uz_i);
693 <          
694 <          duduz_i += preSw * rhat * (ri2 - preRF2_ *  *(idat.rij) );
1026 >        F  += pref * rhat * (trQa * trQb + 2.0 * trQaQb)*v44;
1027 >        F  += pref * rhat * (trQa * rdQbr + trQb * rdQar + 4.0 * rQaQbr)*v45;
1028 >        F  += pref * rhat * (rdQar * rdQbr)*v46;
1029  
1030 <          // Even if we excluded this pair from direct interactions,
1031 <          // we still have the reaction-field-mediated charge-dipole
1032 <          // interaction:
1030 >        F  += pref * 2.0 * (trQb*rQa + trQa*rQb)*v44;
1031 >        F  += pref * 4.0 * (rQaQb + QaQbr)*v44;
1032 >        F  += pref * 2.0 * (rQa*rdQbr + rdQar*rQb)*v45;
1033  
1034 <          if (idat.excluded) {
1035 <            indirect_vpair += -pref * ct_i * preRF2_ * *(idat.rij);
1036 <            indirect_Pot += -preSw * ct_i * preRF2_ * *(idat.rij);
1037 <            indirect_dVdr += -preSw * preRF2_ * uz_i;
1038 <            indirect_duduz_i += -preSw * rhat * preRF2_ *  *(idat.rij);
705 <          }
706 <            
707 <        } else {
708 <          
709 <          // determine inverse r if we are using split dipoles
710 <          if (i_is_SplitDipole) {
711 <            BigR = sqrt( *(idat.r2) + 0.25 * d_i * d_i);
712 <            ri = 1.0 / BigR;
713 <            scale =  *(idat.rij)  * ri;
714 <          } else {
715 <            ri = riji;
716 <            scale = 1.0;
717 <          }
718 <          
719 <          sc2 = scale * scale;
720 <            
721 <          if (screeningMethod_ == DAMPED) {
722 <            // assemble the damping variables
723 <            //res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
724 <            //erfcVal = res.first;
725 <            //derfcVal = res.second;
726 <            erfcVal = erfc(dampingAlpha_ * *(idat.rij));
727 <            derfcVal = - alphaPi_ * exp(-alpha2_ * *(idat.r2));
728 <            c1 = erfcVal * ri;
729 <            c2 = (-derfcVal + c1) * ri;
730 <            c3 = -2.0 * derfcVal * alpha2_ + 3.0 * c2 * ri;
731 <          } else {
732 <            c1 = ri;
733 <            c2 = c1 * ri;
734 <            c3 = 3.0 * c2 * ri;
735 <          }
736 <          
737 <          c2ri = c2 * ri;
738 <              
739 <          // calculate the potential
740 <          pot_term = c2 * scale;
741 <          vterm = pref * ct_i * pot_term;
742 <          vpair += vterm;
743 <          epot +=  *(idat.sw)  * vterm;
1034 >        Ta += pref * (- 4.0 * QaxQb * v41);
1035 >        Ta += pref * (- 2.0 * trQb * cross(rQa, rhat)
1036 >                      + 4.0 * cross(rhat, QaQbr)
1037 >                      - 4.0 * rQaxQbr) * v42;
1038 >        Ta += pref * 2.0 * cross(rhat,Qar) * rdQbr * v43;
1039  
745          // calculate derivatives for the forces and torques
746          dVdr += preSw * (uz_i * c2ri - ct_i * rhat * sc2 * c3);
747          duduz_i += preSw * pot_term * rhat;
748        }
749      }
1040  
1041 <      if (j_is_Dipole) {
1042 <        // variables used by all methods
1043 <        ct_ij = dot(uz_i, uz_j);
1041 >        Tb += pref * (+ 4.0 * QaxQb * v41);
1042 >        Tb += pref * (- 2.0 * trQa * cross(rQb, rhat)
1043 >                      - 4.0 * cross(rQaQb, rhat)
1044 >                      + 4.0 * rQaxQbr) * v42;
1045 >        // Possible replacement for line 2 above:
1046 >        //             + 4.0 * cross(rhat, QbQar)
1047  
1048 <        pref =  *(idat.electroMult) * pre22_ * mu_i * mu_j;
756 <        preSw =  *(idat.sw)  * pref;
1048 >        Tb += pref * 2.0 * cross(rhat,Qbr) * rdQar * v43;
1049  
1050 <        if (summationMethod_ == esm_REACTION_FIELD) {
759 <          ri2 = riji * riji;
760 <          ri3 = ri2 * riji;
761 <          ri4 = ri2 * ri2;
762 <
763 <          vterm = pref * ( ri3 * (ct_ij - 3.0 * ct_i * ct_j) -
764 <                           preRF2_ * ct_ij );
765 <          vpair += vterm;
766 <          epot +=  *(idat.sw)  * vterm;
767 <            
768 <          a1 = 5.0 * ct_i * ct_j - ct_ij;
769 <            
770 <          dVdr += preSw * three * ri4 * (a1 * rhat - ct_i * uz_j - ct_j * uz_i);
771 <
772 <          duduz_i += preSw * (ri3 * (uz_j - three * ct_j * rhat) - preRF2_*uz_j);
773 <          duduz_j += preSw * (ri3 * (uz_i - three * ct_i * rhat) - preRF2_*uz_i);
774 <
775 <          if (idat.excluded) {
776 <            indirect_vpair +=  - pref * preRF2_ * ct_ij;
777 <            indirect_Pot +=    - preSw * preRF2_ * ct_ij;
778 <            indirect_duduz_i += -preSw * preRF2_ * uz_j;
779 <            indirect_duduz_j += -preSw * preRF2_ * uz_i;
780 <          }
781 <
782 <        } else {
783 <          
784 <          if (i_is_SplitDipole) {
785 <            if (j_is_SplitDipole) {
786 <              BigR = sqrt( *(idat.r2) + 0.25 * d_i * d_i + 0.25 * d_j * d_j);
787 <            } else {
788 <              BigR = sqrt( *(idat.r2) + 0.25 * d_i * d_i);
789 <            }
790 <            ri = 1.0 / BigR;
791 <            scale =  *(idat.rij)  * ri;
792 <          } else {
793 <            if (j_is_SplitDipole) {
794 <              BigR = sqrt( *(idat.r2) + 0.25 * d_j * d_j);
795 <              ri = 1.0 / BigR;
796 <              scale =  *(idat.rij)  * ri;
797 <            } else {
798 <              ri = riji;
799 <              scale = 1.0;
800 <            }
801 <          }
802 <          if (screeningMethod_ == DAMPED) {
803 <            // assemble damping variables
804 <            //res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
805 <            //erfcVal = res.first;
806 <            //derfcVal = res.second;
807 <            erfcVal = erfc(dampingAlpha_ * *(idat.rij));
808 <            derfcVal = - alphaPi_ * exp(-alpha2_ * *(idat.r2));
809 <            c1 = erfcVal * ri;
810 <            c2 = (-derfcVal + c1) * ri;
811 <            c3 = -2.0 * derfcVal * alpha2_ + 3.0 * c2 * ri;
812 <            c4 = -4.0 * derfcVal * alpha4_ + 5.0 * c3 * ri * ri;
813 <          } else {
814 <            c1 = ri;
815 <            c2 = c1 * ri;
816 <            c3 = 3.0 * c2 * ri;
817 <            c4 = 5.0 * c3 * ri * ri;
818 <          }
819 <
820 <          // precompute variables for convenience
821 <          sc2 = scale * scale;
822 <          cti3 = ct_i * sc2 * c3;
823 <          ctj3 = ct_j * sc2 * c3;
824 <          ctidotj = ct_i * ct_j * sc2;
825 <          preSwSc = preSw * scale;
826 <          c2ri = c2 * ri;
827 <          c3ri = c3 * ri;
828 <          c4rij = c4 *  *(idat.rij) ;
829 <
830 <          // calculate the potential
831 <          pot_term = (ct_ij * c2ri - ctidotj * c3);
832 <          vterm = pref * pot_term;
833 <          vpair += vterm;
834 <          epot +=  *(idat.sw)  * vterm;
835 <
836 <          // calculate derivatives for the forces and torques
837 <          dVdr += preSwSc * ( ctidotj * rhat * c4rij  -
838 <                              (ct_i*uz_j + ct_j*uz_i + ct_ij*rhat) * c3ri);
839 <          
840 <          duduz_i += preSw * (uz_j * c2ri - ctj3 * rhat);
841 <          duduz_j += preSw * (uz_i * c2ri - cti3 * rhat);
842 <        }
1050 >        //  cerr << " tsum = " << Ta + Tb - cross(  *(idat.d) , F ) << "\n";
1051        }
1052      }
1053  
1054 <    if (i_is_Quadrupole) {
1055 <      if (j_is_Charge) {
1056 <        // precompute some necessary variables
849 <        cx2 = cx_i * cx_i;
850 <        cy2 = cy_i * cy_i;
851 <        cz2 = cz_i * cz_i;
852 <
853 <        pref =  *(idat.electroMult) * pre14_ * q_j * one_third_;
854 <
855 <        if (screeningMethod_ == DAMPED) {
856 <          // assemble the damping variables
857 <          //res = erfcSpline_->getValueAndDerivativeAt( *(idat.rij) );
858 <          //erfcVal = res.first;
859 <          //derfcVal = res.second;
860 <          erfcVal = erfc(dampingAlpha_ * *(idat.rij));
861 <          derfcVal = - alphaPi_ * exp(-alpha2_ * *(idat.r2));
862 <          c1 = erfcVal * riji;
863 <          c2 = (-derfcVal + c1) * riji;
864 <          c3 = -2.0 * derfcVal * alpha2_ + 3.0 * c2 * riji;
865 <          c4 = -4.0 * derfcVal * alpha4_ + 5.0 * c3 * riji * riji;
866 <        } else {
867 <          c1 = riji;
868 <          c2 = c1 * riji;
869 <          c3 = 3.0 * c2 * riji;
870 <          c4 = 5.0 * c3 * riji * riji;
871 <        }
872 <          
873 <        // precompute some variables for convenience
874 <        preSw =  *(idat.sw)  * pref;
875 <        c2ri = c2 * riji;
876 <        c3ri = c3 * riji;
877 <        c4rij = c4 *  *(idat.rij) ;
878 <        rhatdot2 = two * rhat * c3;
879 <        rhatc4 = rhat * c4rij;
880 <
881 <        // calculate the potential
882 <        pot_term = ( qxx_i * (cx2 * c3 - c2ri) +
883 <                     qyy_i * (cy2 * c3 - c2ri) +
884 <                     qzz_i * (cz2 * c3 - c2ri) );
885 <        
886 <        vterm = pref * pot_term;
887 <        vpair += vterm;
888 <        epot +=  *(idat.sw)  * vterm;
889 <
890 <        // calculate the derivatives for the forces and torques
891 <
892 <        dVdr += -preSw * (qxx_i* (cx2*rhatc4 - (two*cx_i*ux_i + rhat)*c3ri) +
893 <                          qyy_i* (cy2*rhatc4 - (two*cy_i*uy_i + rhat)*c3ri) +
894 <                          qzz_i* (cz2*rhatc4 - (two*cz_i*uz_i + rhat)*c3ri));
895 <
896 <        dudux_i += preSw * qxx_i * cx_i *  rhatdot2;
897 <        duduy_i += preSw * qyy_i * cy_i *  rhatdot2;
898 <        duduz_i += preSw * qzz_i * cz_i *  rhatdot2;
899 <      }
1054 >    if (idat.doElectricField) {
1055 >      *(idat.eField1) += Ea * *(idat.electroMult);
1056 >      *(idat.eField2) += Eb * *(idat.electroMult);
1057      }
1058  
1059 +    if (a_is_Fluctuating) *(idat.dVdFQ1) += dUdCa * *(idat.sw);
1060 +    if (b_is_Fluctuating) *(idat.dVdFQ2) += dUdCb * *(idat.sw);
1061  
1062      if (!idat.excluded) {
904      *(idat.vpair) += vpair;
905      (*(idat.pot))[ELECTROSTATIC_FAMILY] += epot;
906      *(idat.f1) += dVdr;
1063        
1064 <      if (i_is_Dipole || i_is_Quadrupole)
1065 <        *(idat.t1) -= cross(uz_i, duduz_i);
1066 <      if (i_is_Quadrupole) {
911 <        *(idat.t1) -= cross(ux_i, dudux_i);
912 <        *(idat.t1) -= cross(uy_i, duduy_i);
913 <      }
1064 >      *(idat.vpair) += U;
1065 >      (*(idat.pot))[ELECTROSTATIC_FAMILY] += U * *(idat.sw);
1066 >      *(idat.f1) += F * *(idat.sw);
1067        
1068 <      if (j_is_Dipole || j_is_Quadrupole)
1069 <        *(idat.t2) -= cross(uz_j, duduz_j);
917 <      if (j_is_Quadrupole) {
918 <        *(idat.t2) -= cross(uz_j, dudux_j);
919 <        *(idat.t2) -= cross(uz_j, duduy_j);
920 <      }
1068 >      if (a_is_Dipole || a_is_Quadrupole)
1069 >        *(idat.t1) += Ta * *(idat.sw);
1070  
1071 +      if (b_is_Dipole || b_is_Quadrupole)
1072 +        *(idat.t2) += Tb * *(idat.sw);
1073 +      
1074      } else {
1075  
1076        // only accumulate the forces and torques resulting from the
1077        // indirect reaction field terms.
1078  
1079 <      *(idat.vpair) += indirect_vpair;
1080 <      (*(idat.pot))[ELECTROSTATIC_FAMILY] += indirect_Pot;
1081 <      *(idat.f1) += indirect_dVdr;
1079 >      *(idat.vpair) += indirect_Pot;      
1080 >      (*(idat.excludedPot))[ELECTROSTATIC_FAMILY] +=  excluded_Pot;
1081 >      (*(idat.pot))[ELECTROSTATIC_FAMILY] += *(idat.sw) * indirect_Pot;
1082 >      *(idat.f1) += *(idat.sw) * indirect_F;
1083        
1084 <      if (i_is_Dipole)
1085 <        *(idat.t1) -= cross(uz_i, indirect_duduz_i);
1086 <      if (j_is_Dipole)
1087 <        *(idat.t2) -= cross(uz_j, indirect_duduz_j);
1084 >      if (a_is_Dipole || a_is_Quadrupole)
1085 >        *(idat.t1) += *(idat.sw) * indirect_Ta;
1086 >            
1087 >      if (b_is_Dipole || b_is_Quadrupole)
1088 >        *(idat.t2) += *(idat.sw) * indirect_Tb;
1089      }
936
937
1090      return;
1091    }  
1092      
1093    void Electrostatic::calcSelfCorrection(SelfData &sdat) {
1094 <    RealType mu1, preVal, chg1, self;
943 <    
1094 >
1095      if (!initialized_) initialize();
1096  
1097      ElectrostaticAtomData data = ElectrostaticMap[sdat.atype];
1098 <  
1098 >    
1099      // logicals
1100      bool i_is_Charge = data.is_Charge;
1101      bool i_is_Dipole = data.is_Dipole;
1102 +    bool i_is_Fluctuating = data.is_Fluctuating;
1103 +    RealType C_a = data.fixedCharge;  
1104 +    RealType self, preVal, DadDa;
1105 +    
1106 +    if (i_is_Fluctuating) {
1107 +      C_a += *(sdat.flucQ);
1108 +      // dVdFQ is really a force, so this is negative the derivative
1109 +      *(sdat.dVdFQ) -=  *(sdat.flucQ) * data.hardness + data.electronegativity;
1110 +      (*(sdat.excludedPot))[ELECTROSTATIC_FAMILY] += (*sdat.flucQ) *
1111 +        (*(sdat.flucQ) * data.hardness * 0.5 + data.electronegativity);
1112 +    }
1113  
1114 <    if (summationMethod_ == esm_REACTION_FIELD) {
1115 <      if (i_is_Dipole) {
1116 <        mu1 = data.dipole_moment;          
1117 <        preVal = pre22_ * preRF2_ * mu1 * mu1;
1118 <        (*(sdat.pot))[ELECTROSTATIC_FAMILY] -= 0.5 * preVal;
1119 <        
1120 <        // The self-correction term adds into the reaction field vector
1121 <        Vector3d uz_i = sdat.eFrame->getColumn(2);
1122 <        Vector3d ei = preVal * uz_i;
1114 >    switch (summationMethod_) {
1115 >    case esm_REACTION_FIELD:
1116 >      
1117 >      if (i_is_Charge) {
1118 >        // Self potential [see Wang and Hermans, "Reaction Field
1119 >        // Molecular Dynamics Simulation with Friedman’s Image Charge
1120 >        // Method," J. Phys. Chem. 99, 12001-12007 (1995).]
1121 >        preVal = pre11_ * preRF_ * C_a * C_a;
1122 >        (*(sdat.pot))[ELECTROSTATIC_FAMILY] -= 0.5 * preVal / cutoffRadius_;
1123 >      }
1124  
1125 <        // This looks very wrong.  A vector crossed with itself is zero.
1126 <        *(sdat.t) -= cross(uz_i, ei);
1125 >      if (i_is_Dipole) {
1126 >        DadDa = data.dipole.lengthSquare();
1127 >        (*(sdat.pot))[ELECTROSTATIC_FAMILY] -= pre22_ * preRF_ * DadDa;
1128        }
1129 <    } else if (summationMethod_ == esm_SHIFTED_FORCE || summationMethod_ == esm_SHIFTED_POTENTIAL) {
1129 >      
1130 >      break;
1131 >      
1132 >    case esm_SHIFTED_FORCE:
1133 >    case esm_SHIFTED_POTENTIAL:
1134        if (i_is_Charge) {        
1135 <        chg1 = data.charge;
968 <        if (screeningMethod_ == DAMPED) {
969 <          self = - 0.5 * (c1c_ + alphaPi_) * chg1 * (chg1 + *(sdat.skippedCharge)) * pre11_;
970 <        } else {        
971 <          self = - 0.5 * rcuti_ * chg1 * (chg1 +  *(sdat.skippedCharge)) * pre11_;
972 <        }
1135 >        self = -0.5 * selfMult_ * C_a * (C_a + *(sdat.skippedCharge)) * pre11_;
1136          (*(sdat.pot))[ELECTROSTATIC_FAMILY] += self;
1137        }
1138 +      break;
1139 +    default:
1140 +      break;
1141      }
1142    }
1143 <
1143 >  
1144    RealType Electrostatic::getSuggestedCutoffRadius(pair<AtomType*, AtomType*> atypes) {
1145      // This seems to work moderately well as a default.  There's no
1146      // inherent scale for 1/r interactions that we can standardize.

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines