Changeset 4dfe724

Timestamp:

Jul 3, 2017, 3:00:00 PM (7 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Action_Thermostats, Add_SelectAtomByNameAction, Adding_Graph_to_ChangeBondActions, Adding_MD_integration_tests, Adding_StructOpt_integration_tests, AutomationFragmentation_failures, Candidate_v1.6.1, ChemicalSpaceEvaluator, EmpiricalPotential_contain_HomologyGraph_documentation, Enhance_userguide, Enhanced_StructuralOptimization, Enhanced_StructuralOptimization_continued, Example_ManyWaysToTranslateAtom, Exclude_Hydrogens_annealWithBondGraph, Fix_Verbose_Codepatterns, ForceAnnealing_oldresults, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_continued, ForceAnnealing_with_BondGraph_continued_betteresults, ForceAnnealing_with_BondGraph_contraction-expansion, Gui_displays_atomic_force_velocity, IndependentFragmentGrids_IntegrationTest, JobMarket_RobustOnKillsSegFaults, JobMarket_StableWorkerPool, PythonUI_with_named_parameters, QtGui_reactivate_TimeChanged_changes, Recreated_GuiChecks, StoppableMakroAction, TremoloParser_IncreasedPrecision, TremoloParser_MultipleTimesteps

Children:

f1cacb

Parents:

70fee0

git-author:

Frederik Heber <heber@…> (04/07/17 17:49:31)

git-committer:

Frederik Heber <frederik.heber@…> (07/03/17 15:00:00)

Message:

FIX: QtGui again reacts to WorldTime changes.

• QtObservedAtom now observes WorldTime and uses new forceUpdate() for position and molecule ObservedValues to get information of new time step.
• Note: As WorldTime is a singleton living till the end of the program we do not need any fancy taking care of subjectKilled() in a correct manner.

File:

• src/UIElements/Qt4/InstanceBoard/QtObservedAtom.cpp (modified) (4 diffs)

src/UIElements/Qt4/InstanceBoard/QtObservedAtom.cpp

@@ -47 +47 @@
 #include "Element/element.hpp"
 #include "World.hpp"
+#include "WorldTime.hpp"
 
 #include "UIElements/Qt4/InstanceBoard/ObservedValue_wCallback.hpp"
@@ -264 +265 @@
 {
   // ObservedValues have been updated before, hence convert updates to Qt's signals
-  switch (notification->getChannelNo()) {
-    case AtomObservable::IndexChanged:
-    {
-      emit indexChanged();
-      break;
+  if (publisher == WorldTime::getPointer()) {
+    // force update of all changes that may occur from one time step to another
+    boost::any_cast<ObservedValue_wCallback<Vector, ObservedValue_Index_t> *>(
+        ObservedValues[AtomPosition])->forceUpdate();
+    boost::any_cast<ObservedValue_wCallback<QtObservedMolecule*, ObservedValue_Index_t> *>(
+        ObservedValues[MoleculeRef])->forceUpdate();
+    // then tell Qt part about updates
+    emit positionChanged();
+    emit moleculeChanged();
+  } else {
+    switch (notification->getChannelNo()) {
+      case AtomObservable::IndexChanged:
+      {
+        emit indexChanged();
+        break;
+      }
+      case AtomObservable::BondsAdded:
+      case AtomObservable::BondsRemoved:
+        emit bondsChanged();
+        break;
+      case AtomObservable::ElementChanged:
+        emit elementChanged();
+        break;
+      case AtomObservable::MoleculeChanged:
+        emit moleculeChanged();
+        break;
+      case AtomObservable::NameChanged:
+        emit nameChanged();
+        break;
+      case AtomObservable::PositionChanged:
+        emit positionChanged();
+        break;
+      case AtomObservable::SelectionChanged:
+        emit selectedChanged();
+        break;
+      default:
+        ASSERT(0, "QtObservedAtom::recieveNotification() - we are not signed on to channel "
+            +toString(notification->getChannelNo())+" of the atom.");
+        break;
     }
-    case AtomObservable::BondsAdded:
-    case AtomObservable::BondsRemoved:
-      emit bondsChanged();
-      break;
-    case AtomObservable::ElementChanged:
-      emit elementChanged();
-      break;
-    case AtomObservable::MoleculeChanged:
-      emit moleculeChanged();
-      break;
-    case AtomObservable::NameChanged:
-      emit nameChanged();
-      break;
-    case AtomObservable::PositionChanged:
-      emit positionChanged();
-      break;
-    case AtomObservable::SelectionChanged:
-      emit selectedChanged();
-      break;
-    default:
-      ASSERT(0, "QtObservedAtom::recieveNotification() - we are not signed on to channel "
-          +toString(notification->getChannelNo())+" of the atom.");
-      break;
   }
 }
@@ -309 +321 @@
   } else
     signedOffChannels = AllsignedOnChannels;
+
+  WorldTime::getInstance().signOn(this, WorldTime::TimeChanged);
 }
 
@@ -324 +338 @@
       board.markObservedAtomAsDisconnected(getIndex());
   }
+
+  WorldTime::getInstance().signOff(this, WorldTime::TimeChanged);
 }