Changeset 07d4b1 for src/Dynamics/BondVectors.cpp

Timestamp:

Apr 10, 2018, 6:43:30 AM (7 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

AutomationFragmentation_failures, Candidate_v1.6.1, ChemicalSpaceEvaluator, Exclude_Hydrogens_annealWithBondGraph, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_contraction-expansion, Gui_displays_atomic_force_velocity, PythonUI_with_named_parameters, StoppableMakroAction, TremoloParser_IncreasedPrecision

Children:

90050b

Parents:

f433ec

git-author:

Frederik Heber <frederik.heber@…> (08/02/17 20:25:57)

git-committer:

Frederik Heber <frederik.heber@…> (04/10/18 06:43:30)

Message:

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

• No more default anneal() without bondgraph, and can't store remnant gradient in force.

File:

• src/Dynamics/BondVectors.cpp (modified) (4 diffs)

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;
 }