Changeset ccbdf4

Timestamp:

Aug 2, 2017, 8:25:57 PM (8 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

ForceAnnealing_with_BondGraph_continued

Children:

c4a725

Parents:

90e540

Message:

tempcommit: BondVectors::getRemnant...() now requires atom's gradient.

Location:

src/Dynamics

Files:

• BondVectors.cpp (modified) (4 diffs)
• BondVectors.hpp (modified) (2 diffs)
• ForceAnnealing.hpp (modified) (1 diff)

src/Dynamics/BondVectors.cpp

@@ -287 +287 @@
 Vector BondVectors::getRemnantGradientForAtomAtStep(
     const atom &_walker,
+    const Vector &_walkerGradient,
     const std::vector<Vector> _BondVectors,
     const BondVectors::weights_t &_weights,
@@ -292 +293 @@
     forcestore_t _forcestore) const
 {
-  const Vector &walkerGradient = _walker.getAtomicForceAtStep(_step);
   BondVectors::weights_t::const_iterator weightiter = _weights.begin();
   std::vector<Vector>::const_iterator vectoriter = _BondVectors.begin();
@@ -303 +303 @@
     const Vector &BondVector = *vectoriter;
 
-    const double temp = (*weightiter)*walkerGradient.ScalarProduct(BondVector);
+    const double temp = (*weightiter)*_walkerGradient.ScalarProduct(BondVector);
     _forcestore(_walker, current_bond, _step, temp);
     LOG(4, "DEBUG: BondVector " << BondVector << " receives projected force of "
-        << temp);
+        << (*weightiter) << "*" << _walkerGradient << "*" << BondVector << " = " << temp);
     forcesum += temp * BondVector;
   }
@@ -314 +314 @@
       "BondVectors::getRemnantGradientForAtomAtStep() - vectoriter is not at end when it should be.");
 
-  return walkerGradient-forcesum;
+  return _walkerGradient-forcesum;
 }
 

src/Dynamics/BondVectors.hpp

@@ -129 +129 @@
    *
    * \param _walker atom to get BondVectors for
+   * \param _walkerGradient gradient of atom to get BondVectors for
    * \param _BondVectors precalculated bond vectors for given \a _walker
    * \param _weights weight per bond vector (as it is a frame, not a basis)
@@ -137 +138 @@
   Vector getRemnantGradientForAtomAtStep(
       const atom &_walker,
+      const Vector &_walkerGradient,
       const std::vector<Vector> _BondVectors,
       const BondVectors::weights_t &_weights,

src/Dynamics/ForceAnnealing.hpp

@@ -352 +352 @@
                   boost::cref(bv), boost::ref(projected_forces));
           const Vector RemnantGradient = bv.getRemnantGradientForAtomAtStep(
-              walker, BondVectors, weights, timestep, forcestoring
+              walker, walkerGradient, BondVectors, weights, timestep, forcestoring
           );
           RemnantGradient_per_atom.insert( std::make_pair(walker.getId(), RemnantGradient) );