Changeset 4919f0 for src/Actions

Timestamp:

Aug 3, 2017, 10:46:48 AM (8 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

ForceAnnealing_with_BondGraph_continued

Children:

cee565

Parents:

fcc860

Message:

atom::removeStep() extended to removing interval of steps.

• AtomInfo::removeTrajectorySteps() and BondedParticle::removeTrajectorySteps() changed and all calls adapted.

Location:

src/Actions

Files:

• MoleculeAction/VerletIntegrationAction.cpp (modified) (1 diff)
• UndoRedoHelpers.cpp (modified) (1 diff)
• UndoRedoHelpers.hpp (modified) (1 diff)

src/Actions/MoleculeAction/VerletIntegrationAction.cpp

@@ -134 +134 @@
 
   // remove current step for all modified atoms
-  removeLastStep(getIdsFromAtomicInfo(state->UndoInfo), CurrentStep+1);
+  removeSteps(getIdsFromAtomicInfo(state->UndoInfo), CurrentStep+1, CurrentStep+1);
 
   // and set back the old step (forces have been changed)

src/Actions/UndoRedoHelpers.cpp

@@ -219 +219 @@
 }
 
-void MoleCuilder::removeLastStep(const std::vector<atomId_t> &_atoms, const unsigned int _step)
-{
-  for (size_t i=0; i<_atoms.size(); ++i) {
-    atom * const _atom = World::getInstance().getAtom(AtomById(_atoms[i]));
-    _atom->removeStep(_step);
+void MoleCuilder::removeSteps(
+    const std::vector<atomId_t> &movedatoms,
+    const unsigned int _firststep,
+    const unsigned int _laststep)
+{
+  for (std::vector<atomId_t>::const_iterator iter = movedatoms.begin();
+      iter != movedatoms.end(); ++iter) {
+    atom * const _atom = World::getInstance().getAtom(AtomById(*iter));
+    _atom->removeSteps(_firststep, _laststep);
   }
 }

src/Actions/UndoRedoHelpers.hpp

@@ -111 +111 @@
   void RemoveMoleculesWithAtomsByIds(const std::vector<moleculeId_t> &ids);
 
-  /** Removes the last time step for all \a movedatoms
+  /** Removes the time steps in given interval for all \a movedatoms.
    *
-   * @param movedatoms atoms whose last step in time to remove
-   * @param _step which trajectory to remove
+   * @param movedatoms atoms whose steps in [\a _firststep, \a _laststep] in time to remove
+   * @param _firststep first step in interval to be removed
+   * @param _laststep last step in interval to be removed
    */
-  void removeLastStep(const std::vector<atomId_t> &movedatoms, const unsigned int _step);
+  void removeSteps(
+      const std::vector<atomId_t> &movedatoms,
+      const unsigned int _firststep,
+      const unsigned int _laststep);
 
   /** Adds another time step to all \a movedatoms.