Changeset 534374 for src

Timestamp:

Aug 5, 2015, 5:32:07 PM (9 years ago)

Author:

Frederik Heber <heber@…>

Branches:

Action_Thermostats, Add_AtomRandomPerturbation, Add_FitFragmentPartialChargesAction, Add_RotateAroundBondAction, Add_SelectAtomByNameAction, Added_ParseSaveFragmentResults, AddingActions_SaveParseParticleParameters, Adding_Graph_to_ChangeBondActions, Adding_MD_integration_tests, Adding_ParticleName_to_Atom, Adding_StructOpt_integration_tests, AtomFragments, Automaking_mpqc_open, AutomationFragmentation_failures, Candidate_v1.5.4, Candidate_v1.6.0, Candidate_v1.6.1, ChangeBugEmailaddress, ChangingTestPorts, ChemicalSpaceEvaluator, CombiningParticlePotentialParsing, Combining_Subpackages, Debian_Package_split, Debian_package_split_molecuildergui_only, Disabling_MemDebug, Docu_Python_wait, EmpiricalPotential_contain_HomologyGraph, EmpiricalPotential_contain_HomologyGraph_documentation, Enable_parallel_make_install, Enhance_userguide, Enhanced_StructuralOptimization, Enhanced_StructuralOptimization_continued, Example_ManyWaysToTranslateAtom, Exclude_Hydrogens_annealWithBondGraph, FitPartialCharges_GlobalError, Fix_BoundInBox_CenterInBox_MoleculeActions, Fix_ChargeSampling_PBC, Fix_ChronosMutex, Fix_FitPartialCharges, Fix_FitPotential_needs_atomicnumbers, Fix_ForceAnnealing, Fix_IndependentFragmentGrids, Fix_ParseParticles, Fix_ParseParticles_split_forward_backward_Actions, Fix_PopActions, Fix_QtFragmentList_sorted_selection, Fix_Restrictedkeyset_FragmentMolecule, Fix_StatusMsg, Fix_StepWorldTime_single_argument, Fix_Verbose_Codepatterns, Fix_fitting_potentials, Fixes, ForceAnnealing_goodresults, ForceAnnealing_oldresults, ForceAnnealing_tocheck, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_continued, ForceAnnealing_with_BondGraph_continued_betteresults, ForceAnnealing_with_BondGraph_contraction-expansion, FragmentAction_writes_AtomFragments, FragmentMolecule_checks_bonddegrees, GeometryObjects, Gui_Fixes, Gui_displays_atomic_force_velocity, ImplicitCharges, IndependentFragmentGrids, IndependentFragmentGrids_IndividualZeroInstances, IndependentFragmentGrids_IntegrationTest, IndependentFragmentGrids_Sole_NN_Calculation, JobMarket_RobustOnKillsSegFaults, JobMarket_StableWorkerPool, JobMarket_unresolvable_hostname_fix, MoreRobust_FragmentAutomation, ODR_violation_mpqc_open, PartialCharges_OrthogonalSummation, PdbParser_setsAtomName, PythonUI_with_named_parameters, QtGui_reactivate_TimeChanged_changes, Recreated_GuiChecks, Rewrite_FitPartialCharges, RotateToPrincipalAxisSystem_UndoRedo, SaturateAtoms_findBestMatching, SaturateAtoms_singleDegree, StoppableMakroAction, Subpackage_CodePatterns, Subpackage_JobMarket, Subpackage_LinearAlgebra, Subpackage_levmar, Subpackage_mpqc_open, Subpackage_vmg, Switchable_LogView, ThirdParty_MPQC_rebuilt_buildsystem, TrajectoryDependenant_MaxOrder, TremoloParser_IncreasedPrecision, TremoloParser_MultipleTimesteps, TremoloParser_setsAtomName, Ubuntu_1604_changes, stable

Children:

009e2e2

Parents:

b3a33d

git-author:

Frederik Heber <heber@…> (06/19/15 17:37:50)

git-committer:

Frederik Heber <heber@…> (08/05/15 17:32:07)

Message:

GLMoleculeObject_atom uses ObservedValue for internally representing atom's state.

• we use internal owner as ref to Observable.

Location:

src/UIElements/Views/Qt4/Qt3D

Files:

• GLMoleculeObject_atom.cpp (modified) (4 diffs)
• GLMoleculeObject_atom.hpp (modified) (4 diffs)

src/UIElements/Views/Qt4/Qt3D/GLMoleculeObject_atom.cpp

@@ -44 +44 @@
 #include "CodePatterns/Observer/Notification.hpp"
 
+#include <algorithm>
+#include <boost/assign.hpp>
+
 #include "Atom/atom.hpp"
 #include "Bond/bond.hpp"
 #include "Descriptors/AtomIdDescriptor.hpp"
 #include "Element/element.hpp"
+#include "Element/periodentafel.hpp"
 #include "LinearAlgebra/Vector.hpp"
 #include "GLMoleculeObject_bond.hpp"
@@ -53 +57 @@
 #include "WorldTime.hpp"
 
+using namespace boost::assign;
+
+static const Observable::channels_t getAtomBondsChannels()
+{
+  Observable::channels_t channels;
+  channels += AtomObservable::BondsAdded, AtomObservable::BondsRemoved;
+  return channels;
+}
+
+// static entities
+const Observable::channels_t
+GLMoleculeObject_atom::AtomIndexChannels(1, AtomObservable::IndexChanged);
+const Observable::channels_t
+GLMoleculeObject_atom::AtomPositionChannels(1, AtomObservable::PositionChanged);
+const Observable::channels_t
+GLMoleculeObject_atom::AtomElementChannels(1, AtomObservable::ElementChanged);
+const Observable::channels_t
+GLMoleculeObject_atom::AtomBondsChannels(getAtomBondsChannels());
+const Observable::channels_t
+GLMoleculeObject_atom::AtomSelectionStatusChannels(World::SelectionChanged);
+
 GLMoleculeObject_atom::GLMoleculeObject_atom(QGLSceneNode *mesh[], QObject *parent, const atomId_t _id) :
   GLMoleculeObject(mesh, parent),
   Observer(std::string("GLMoleculeObject_atom")+toString(_id)),
-  atomicid(_id),
-  uptodatePosition(false),
-  uptodateElement(false)
-{
-  // sign on as observer (obtain non-const instance before)
-  const atom * const _atom = const_cast<const World &>(World::getInstance()).
-      getAtom(AtomById(atomicid));
-  if (_atom != NULL) {
-    _atom->signOn(this, AtomObservable::IndexChanged);
-    _atom->signOn(this, AtomObservable::PositionChanged);
-    _atom->signOn(this, AtomObservable::ElementChanged);
-    _atom->signOn(this, AtomObservable::BondsAdded);
-  } else {
-    ELOG(2, "Atom with id "+toString(atomicid)+" is already gone.");
-  }
-  World::getInstance().signOn(this, World::SelectionChanged);
-
-  // set the object's id
-  resetProperties();
-
-  LOG(2, "INFO: Created sphere for atom " << atomicid << ".");
+  atomref(getAtom(_id)),
+  AtomIndex(
+      atomref,
+      boost::bind(&GLMoleculeObject_atom::updateIndex, this),
+      "AtomIndex_"+toString(_id),
+      _id,
+      AtomIndexChannels),
+  AtomPosition(
+      atomref,
+      boost::bind(&GLMoleculeObject_atom::updatePosition, this),
+      "AtomPosition_"+toString(_id),
+      updatePosition(),
+      AtomPositionChannels),
+  AtomElement(
+      atomref,
+      boost::bind(&GLMoleculeObject_atom::updateElement, this),
+      "AtomElement"+toString(_id),
+      updateElement(),
+      AtomElementChannels),
+  AtomBonds(
+      atomref,
+      boost::bind(&GLMoleculeObject_atom::updateBonds, this),
+      "AtomBonds_"+toString(_id),
+      updateBonds(),
+      AtomBondsChannels),
+  AtomSelectionStatus(
+      World::getPointer(),
+      boost::bind(&GLMoleculeObject_atom::updateSelectionStatus, this),
+      "AtomSelectionStatus_"+toString(_id),
+      updateSelectionStatus(),
+      AtomSelectionStatusChannels),
+  owner(NULL)
+{
+  setObjectId(_id);
+  resetPosition();
+  resetElement();
+
+  // sign On
+  activateObserver();
+
+  // atomref is only used for caching the ref, it must be used elswhere
+  const_cast<atom *&>(atomref) = NULL;
 
   connect( this, SIGNAL(clicked()), this, SLOT(wasClicked()));
+  connect( this, SIGNAL(idChanged()), this, SLOT(resetIndex()), Qt::QueuedConnection);
+  connect( this, SIGNAL(elementChanged()), this, SLOT(resetElement()), Qt::QueuedConnection);
+  connect( this, SIGNAL(positionChanged()), this, SLOT(resetPosition()), Qt::QueuedConnection);
+  connect( this, SIGNAL(bondsChanged()), this, SLOT(resetPosition()), Qt::QueuedConnection);
+  connect( this, SIGNAL(selectionstatusChanged()), this, SLOT(resetSelectionStatus()), Qt::QueuedConnection);
+}
+
+void GLMoleculeObject_atom::activateObserver()
+{
+  if (atomref != NULL) {
+    owner = static_cast<const Observable *>(atomref);
+    owner->signOn(this, AtomObservable::IndexChanged);
+    owner->signOn(this, AtomObservable::PositionChanged);
+    owner->signOn(this, AtomObservable::ElementChanged);
+    owner->signOn(this, AtomObservable::BondsAdded);
+    owner->signOn(this, AtomObservable::BondsRemoved);
+    World::getInstance().signOn(this, World::SelectionChanged);
+  }
+}
+
+
+void GLMoleculeObject_atom::deactivateObserver()
+{
+  // sign Off
+  if (owner != NULL) {
+    owner->signOff(this, AtomObservable::IndexChanged);
+    owner->signOff(this, AtomObservable::PositionChanged);
+    owner->signOff(this, AtomObservable::ElementChanged);
+    owner->signOff(this, AtomObservable::BondsAdded);
+    owner->signOff(this, AtomObservable::BondsRemoved);
+    World::getInstance().signOff(this, World::SelectionChanged);
+    owner = NULL;
+  }
 }
 
 GLMoleculeObject_atom::~GLMoleculeObject_atom()
 {
-  const atom * const _atom = const_cast<const World &>(World::getInstance()).
-      getAtom(AtomById(atomicid));
-  if (_atom != NULL){
-    _atom->signOff(this, AtomObservable::IndexChanged);
-    _atom->signOff(this, AtomObservable::PositionChanged);
-    _atom->signOff(this, AtomObservable::ElementChanged);
-    _atom->signOff(this, AtomObservable::BondsAdded);
-  }
-  World::getInstance().signOff(this, World::SelectionChanged);
-}
-
-void GLMoleculeObject_atom::update(Observable *publisher)
-{
-#ifdef LOG_OBSERVER
-  observerLog().addMessage() << "++ Update of Observer " << observerLog().getName(static_cast<Observer *>(this)) << " from atom "+toString(atomicid)+".";
-#endif
-  resetProperties();
+  deactivateObserver();
+}
+
+void GLMoleculeObject_atom::resetIndex()
+{
+  const atomId_t newId = AtomIndex.get();
+  const size_t oldId = objectId();
+  ASSERT( newId != oldId,
+      "GLMoleculeObject_atom::updateIndex() - index "+toString(newId)+" did not change.");
+  LOG(4, "INFO: GLMoleculeObject_atom::resetIndex() - new index is "+toString(newId)+".");
+  setObjectId(newId);
+
+  emit indexChanged(this, oldId, newId);
 }
 
 void GLMoleculeObject_atom::resetPosition()
 {
-  const atom * const _atom = const_cast<const World &>(World::getInstance()).
-      getAtom(AtomById(atomicid));
-  if (_atom != NULL) {
-    const Vector Position = _atom->getPosition();
-    LOG(4, "INFO: GLMoleculeObject_atom::resetIndex() - new position is "+toString(Position)+".");
-    setPosition(QVector3D(Position[0], Position[1], Position[2]));
-  } else {
-    ELOG(2, "Atom with id "+toString(atomicid)+" is already gone.");
-  }
-  uptodatePosition = true;
+  const Vector Position = AtomPosition.get();
+  LOG(4, "INFO: GLMoleculeObject_atom::resetIndex() - new position is "+toString(Position)+".");
+  setPosition(QVector3D(Position[0], Position[1], Position[2]));
 }
 
@@ -119 +187 @@
 {
   size_t elementno = 0;
-  const atom * const _atom = const_cast<const World &>(World::getInstance()).
-      getAtom(AtomById(atomicid));
-  const element *_type = NULL;
-  if (_atom != NULL) {
-    _type = _atom->getType();
-  } else {
-    ELOG(2, "Atom with id "+toString(atomicid)+" is already gone.");
-  }
+  const element * const _type = World::getInstance().
+      getPeriode()->FindElement(AtomElement.get());
   if (_type != NULL) {
     elementno = _type->getAtomicNumber();
@@ -148 +210 @@
   }
   setScale( radius / 4. );
-
-  uptodateElement = true;
-}
-
-void GLMoleculeObject_atom::resetIndex()
-{
-  int oldId = objectId();
-  LOG(4, "INFO: GLMoleculeObject_atom::resetIndex() - new index is "+toString(atomicid)+".");
-  setObjectId(atomicid);
-
-  emit indexChanged(this, oldId, atomicid);
-}
-
-void GLMoleculeObject_atom::resetProperties()
-{
-  // set position
-  resetPosition();
-
-  // set element
-  resetElement();
-
-  // set the object's id
-  resetIndex();
-
-  // selected?
-  setSelected(World::getInstance().isAtomSelected(atomicid));
+}
+
+void GLMoleculeObject_atom::resetBonds()
+{
+  ListOfBonds_t ListOfBonds_new = AtomBonds.get();
+  std::sort(ListOfBonds_new.begin(), ListOfBonds_new.end());
+  ListOfBonds_t BondsToAdd;
+  std::set_difference(
+      ListOfBonds_new.begin(), ListOfBonds_new.end(),
+      ListOfBonds.begin(), ListOfBonds.end(),
+      std::back_inserter(BondsToAdd));
+  ListOfBonds_t BondsToRemove;
+  std::set_difference(
+      ListOfBonds.begin(), ListOfBonds.end(),
+      ListOfBonds_new.begin(), ListOfBonds_new.end(),
+      std::back_inserter(BondsToRemove));
+  for (ListOfBonds_t::const_iterator iter = BondsToAdd.begin();
+      iter != BondsToAdd.end();
+      ++iter) {
+    const GLMoleculeObject_bond::SideOfBond side = (iter->first == AtomIndex.get()) ?
+        GLMoleculeObject_bond::left : GLMoleculeObject_bond::right;
+    emit BondsAdded(iter->first, iter->second, side);
+  }
+  for (ListOfBonds_t::const_iterator iter = BondsToRemove.begin();
+      iter != BondsToRemove.end();
+      ++iter) {
+    emit BondsRemoved(iter->first, iter->second);
+  }
+  ListOfBonds = ListOfBonds_new;
+}
+
+void GLMoleculeObject_atom::resetSelectionStatus()
+{
+  setSelected(AtomSelectionStatus.get());
 }
 
 void GLMoleculeObject_atom::draw(QGLPainter *painter, const QVector4D &cameraPlane)
 {
-  // hook to update prior to painting
-  if (!uptodatePosition)
-    resetPosition();
-  if (!uptodateElement)
-    resetElement();
-
   // call old hook to do the actual paining
   GLMoleculeObject::draw(painter, cameraPlane);
 }
 
+void GLMoleculeObject_atom::wasClicked()
+{
+  LOG(4, "INFO: GLMoleculeObject_atom: atom " << AtomIndex.get() << " has been clicked");
+  emit clicked(AtomIndex.get());
+}
+
+const atom * const GLMoleculeObject_atom::getAtomConst(const atomId_t _id)
+{
+  const atom * const _atom = const_cast<const World &>(World::getInstance()).
+      getAtom(AtomById(_id));
+  return _atom;
+}
+
+atom * const GLMoleculeObject_atom::getAtom(const atomId_t _id)
+{
+  atom * const _atom = World::getInstance().getAtom(AtomById(_id));
+  return _atom;
+}
+
+atomId_t GLMoleculeObject_atom::updateIndex() const
+{
+  const atom * const _atom = World::getInstance().lastChanged<atom>();
+  if (_atom != NULL) {
+    return _atom->getId();
+  }
+  return (atomId_t)-1;
+}
+
+Vector GLMoleculeObject_atom::updatePosition() const
+{
+  const atom * const _atom = getAtom(AtomIndex.get());
+  if (_atom != NULL) {
+    return _atom->getPosition();
+  } else {
+    return zeroVec;
+  }
+}
+
+atomicNumber_t GLMoleculeObject_atom::updateElement() const
+{
+  const atom * const _atom = getAtom(AtomIndex.get());
+  if (_atom != NULL) {
+    return _atom->getElementNo();
+  } else {
+    return (atomicNumber_t)-1;
+  }
+}
+
+GLMoleculeObject_atom::ListOfBonds_t GLMoleculeObject_atom::updateBonds() const
+{
+  ListOfBonds_t ListOfBonds;
+  const atom * const _atom = getAtom(AtomIndex.get());
+  if (_atom != NULL) {
+    // make sure position is up-to-date
+    const BondList ListBonds = _atom->getListOfBonds();
+    for (BondList::const_iterator iter = ListBonds.begin();
+        iter != ListBonds.end();
+        ++iter)
+      ListOfBonds.insert( ListOfBonds.end(), std::make_pair(
+          (*iter)->leftatom->getId(),
+          (*iter)->rightatom->getId()) );
+  } else {
+    ELOG(2, "Atom with id "+toString(AtomIndex.get())+" is already gone.");
+  }
+  return ListOfBonds;
+}
+
+bool GLMoleculeObject_atom::updateSelectionStatus() const
+{
+  const atom * const _atom = getAtom(AtomIndex.get());
+  if (_atom != NULL) {
+    return const_cast<const World &>(World::getInstance()).isSelected(_atom);
+  }
+  return false;
+}
+
+void GLMoleculeObject_atom::update(Observable *publisher)
+{
+  ASSERT(0, "GLMoleculeObject_atom::update() - we are not signed on for global updates.");
+}
+
 void GLMoleculeObject_atom::subjectKilled(Observable *publisher)
 {
-  // remove id such that we don't sign off accidentally from a different atom
-  const_cast<atomId_t &>(atomicid) = -1;
+  deactivateObserver();
 }
 
 void GLMoleculeObject_atom::recieveNotification(Observable *publisher, Notification_ptr notification)
 {
-  const atom * const _atom = const_cast<const World &>(World::getInstance()).
-      getAtom(AtomById(atomicid));
-  if (publisher == dynamic_cast<const Observable*>(_atom)){
-    // notofication from atom
-#ifdef LOG_OBSERVER
-    observerLog().addMessage() << "++ Update of Observer "<< observerLog().getName(static_cast<Observer *>(this))
-          << " received notification from atom " << _atom->getId() << " for channel "
-          << notification->getChannelNo() << ".";
-#endif
+  // ObservedValues have been updated before, hence convert updates to Qt's signals
+  atom * const _atom = dynamic_cast<atom *>(publisher);
+  if (_atom != NULL) {
     switch (notification->getChannelNo()) {
+      case AtomObservable::IndexChanged:
+        emit idChanged();
+        break;
+      case AtomObservable::PositionChanged:
+        emit positionChanged();
+        break;
       case AtomObservable::ElementChanged:
-        uptodateElement = false;
-        break;
-      case AtomObservable::IndexChanged:
-        resetIndex();
-        break;
-      case AtomObservable::PositionChanged:
-        uptodatePosition = false;
+        emit elementChanged();
         break;
       case AtomObservable::BondsAdded:
-      {
-        const atom * const _atom = const_cast<const World &>(World::getInstance()).
-            getAtom(AtomById(atomicid));
-        if (_atom != NULL) {
-          // make sure position is up-to-date
-          if (!uptodatePosition)
-            resetPosition();
-          ASSERT(!_atom->getListOfBonds().empty(),
-              "GLMoleculeObject_atom::recieveNotification() - received BondsAdded but ListOfBonds is empty.");
-          const bond::ptr _bond = *(_atom->getListOfBonds().rbegin());
-          const GLMoleculeObject_bond::SideOfBond side = (_bond->leftatom == _atom) ?
-              GLMoleculeObject_bond::left : GLMoleculeObject_bond::right;
-          emit BondsInserted(_bond, side);
-        } else {
-          ELOG(2, "Atom with id "+toString(atomicid)+" is already gone.");
-        }
-        break;
-      }
+      case AtomObservable::BondsRemoved:
+        emit bondsChanged();
+        break;
       default:
-        //setProperties();
+        ASSERT(0, "GLMoleculeObject_atom::recieveNotification() - we are not signed on to channel "
+            +toString(notification->getChannelNo())+" of the atom.");
         break;
     }
-  }else if (static_cast<World *>(publisher) == World::getPointer()) {
-    // notification from world
-#ifdef LOG_OBSERVER
-    observerLog().addMessage() << "++ Update of Observer "<< observerLog().getName(static_cast<Observer *>(this))
-          << " received notification from world for channel "
-          << notification->getChannelNo() << ".";
-#endif
+  } else if (publisher == static_cast<World *>(World::getPointer())) {
     switch (notification->getChannelNo()) {
       case World::SelectionChanged:
-        if (_atom != NULL)
-          setSelected(World::getInstance().isSelected(_atom));
+        emit selectionstatusChanged();
         break;
       default:
+        ASSERT(0, "GLMoleculeObject_atom::recieveNotification() - we are not signed on to channel "
+            +toString(notification->getChannelNo())+" of the World.");
         break;
     }
   }
 }
-
-void GLMoleculeObject_atom::wasClicked()
-{
-  LOG(4, "INFO: GLMoleculeObject_atom: atom " << atomicid << " has been clicked");
-  emit clicked(atomicid);
-}

src/UIElements/Views/Qt4/Qt3D/GLMoleculeObject_atom.hpp

@@ -16 +16 @@
 #include "GLMoleculeObject.hpp"
 
+#include "CodePatterns/ObservedValue.hpp"
 #include "CodePatterns/Observer/Observer.hpp"
+
+#include "LinearAlgebra/Vector.hpp"
 
 #include "Bond/bond.hpp"
@@ -24 +27 @@
 class GLWorldScene;
 
-class GLMoleculeObject_atom : public GLMoleculeObject, public Observer
+class GLMoleculeObject_atom : public GLMoleculeObject, Observer
 {
   Q_OBJECT
@@ -40 +43 @@
 private slots:
   void wasClicked();
+  void resetIndex();
+  void resetElement();
+  void resetPosition();
+  void resetBonds();
+  void resetSelectionStatus();
 
 signals:
   void clicked(atomId_t);
-  void BondsInserted(const bond::ptr _bond, const GLMoleculeObject_bond::SideOfBond side);
-  void indexChanged(GLMoleculeObject_atom *ob, int oldId, int newId);
+  void BondsAdded(const atomId_t _left, const atomId_t _right, const GLMoleculeObject_bond::SideOfBond side);
+  void BondsRemoved(const atomId_t _left, const atomId_t _right);
+  void indexChanged(GLMoleculeObject_atom *ob, const atomId_t oldId, const atomId_t newId);
+  void idChanged();
+  void positionChanged();
+  void elementChanged();
+  void bondsChanged();
+  void selectionstatusChanged();
 
 private:
@@ -50 +64 @@
   friend class GLMoleculeObject_molecule;
 
-  void resetPosition();
+  //!> typedef for list of bonds, defined by pairs of atom ids
+  typedef std::vector< std::pair<atomId_t, atomId_t> > ListOfBonds_t;
+
+  static const atom * const getAtomConst(const atomId_t _id);
+  static atom * const getAtom(const atomId_t _id);
+
+  atomId_t updateIndex() const;
+  Vector updatePosition() const;
+  atomicNumber_t updateElement() const;
+  ListOfBonds_t updateBonds() const;
+  bool updateSelectionStatus() const;
+
+  void activateObserver();
+  void deactivateObserver();
 
 private:
-  void resetElement();
-  void resetIndex();
-  void resetProperties();
 
-  const atomId_t atomicid;
+  //!> current list of bonds to compare new onw against for changes
+  ListOfBonds_t ListOfBonds;
 
-  bool uptodatePosition;
-  bool uptodateElement;
+  //!> temporary variable used in cstor
+  atom * const atomref;
+
+  //!> cached value of the atom's id
+  ObservedValue<atomId_t> AtomIndex;
+  //!> cached value of the atom's position
+  ObservedValue<Vector> AtomPosition;
+  //!> cached value of the atom's element
+  ObservedValue<atomicNumber_t> AtomElement;
+  //!> cached value of the atom's id
+  ObservedValue<ListOfBonds_t> AtomBonds;
+  //!> cached value of the atom's selection status
+  ObservedValue<bool> AtomSelectionStatus;
+
+  //!> list of channels when index needs to update
+  static const Observable::channels_t AtomIndexChannels;
+  //!> list of channels when position needs to update
+  static const Observable::channels_t AtomPositionChannels;
+  //!> list of channels when element needs to update
+  static const Observable::channels_t AtomElementChannels;
+  //!> list of channels when bonds needs to update
+  static const Observable::channels_t AtomBondsChannels;
+  //!> list of channels when selection status needs to update
+  static const Observable::channels_t AtomSelectionStatusChannels;
+
+  //!> the Observable we are signed on, also indicates whether we are sign on (not NULL)
+  const Observable *owner;
 };