Changeset 59eabc for src/UIElements/Qt4/InstanceBoard/QtObservedInstanceBoard.cpp

Timestamp:

Apr 20, 2016, 7:17:41 AM (10 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, Candidate_v1.7.0, Candidate_v1.7.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:

1a58ce

Parents:

bff202

git-author:

Frederik Heber <heber@…> (03/21/16 20:42:05)

git-committer:

Frederik Heber <heber@…> (04/20/16 07:17:41)

Message:

Introduced unique and permanent ObservedValue_Index_t.

• this is currently the address of the observable which is not perfectly secure but might work for the moment.
• QtObservedInstanceBoard uses the index internally and only has an additional map to serve instances on request for the their atomId_t, moleculeId_t, .... However, this id should nowhere else be used because it may change during the lifetime of the object.
• this should prevent the hazzle with the possibly changing atomId_t, moleculeId_t, ... in all classes that work with QtObserved... instances.

File:

• src/UIElements/Qt4/InstanceBoard/QtObservedInstanceBoard.cpp (modified) (6 diffs)

src/UIElements/Qt4/InstanceBoard/QtObservedInstanceBoard.cpp

@@ -67 +67 @@
       boost::bind(&QtObservedInstanceBoard::moleculecountsubjectKilled, this, _1))
 {
+  qRegisterMetaType<ObservedValue_Index_t>("ObservedValue_Index_t");
   qRegisterMetaType<QtObservedAtom::ptr>("QtObservedAtom::ptr");
   qRegisterMetaType<QtObservedMolecule::ptr>("QtObservedMolecule::ptr");
@@ -141 +142 @@
                   *this));
           observedmolecule->setSelfRef(observedmolecule);
-          moleculeObservedValues.insert(_id, observedmolecule);
+          const ObservedValue_Index_t index = static_cast<const Observable *>(_molecule);
+          moleculeObservedValues.insert(index, observedmolecule);
+          moleculeids_lookup.insert( std::make_pair(_id, index) );
           // we need to check for index changes
           LOG(3, "DEBUG: InformationBoard signOn()s to molecule " << _id);
@@ -167 +170 @@
               new QtObservedAtom(_id, _atom, *this));
           observedatom->setSelfRef(observedatom);
-          atomObservedValues.insert(_id, observedatom);
+          const ObservedValue_Index_t index = static_cast<const Observable *>(_atom);
+          atomObservedValues.insert(index, observedatom);
+          atomids_lookup.insert( std::make_pair(_id, index) );
           // we need to check for index changes
           LOG(3, "DEBUG: InformationBoard signOn()s to atom " << _id);
@@ -190 +195 @@
         const moleculeId_t newmoleculeId = dynamic_cast<molecule *>(publisher)->getId();
         LOG(3, "DEBUG: InformationBoard got IndexChanged from molecule " << molid << " to " << newmoleculeId);
-#ifndef NDEBUG
-        bool status =
-#endif
-        moleculeObservedValues.changeIdentifier(molid, newmoleculeId);
-        ASSERT( status,
-            "QtObservedInstanceBoard::recieveNotification() - cannot change molecule's id "
-            +toString(molid)+" "+toString(newmoleculeId)+" in moleculeObservedValues.");
+        {
+          typename IdtoIndex_t<moleculeId_t>::iterator iter = moleculeids_lookup.find(molid);
+          ASSERT( iter != moleculeids_lookup.end(),
+              "QtObservedInstanceBoard::recieveNotification() - mol id "
+              +toString(molid)+" unknown to lookup.");
+          const ObservedValue_Index_t index = iter->second;
+          moleculeids_lookup.erase(iter);
+          moleculeids_lookup.insert( std::pair<moleculeId_t, ObservedValue_Index_t>(newmoleculeId, index) );
+        }
         // no need update SignedOn, ref does not change
         emit moleculeIndexChanged(molid, newmoleculeId);
@@ -212 +219 @@
         const atomId_t newatomId = dynamic_cast<atom *>(publisher)->getId();
         LOG(3, "DEBUG: InformationBoard got IndexChanged from atom " << oldatomId << " to " << newatomId);
-#ifndef NDEBUG
-        bool status =
-#endif
-        atomObservedValues.changeIdentifier(oldatomId, newatomId);
-        ASSERT( status,
-            "QtObservedInstanceBoard::recieveNotification() - cannot change atom's id "
-            +toString(oldatomId)+" "+toString(newatomId)+" in atomObservedValues.");
+        {
+          typename IdtoIndex_t<atomId_t>::iterator iter = atomids_lookup.find(oldatomId);
+          ASSERT( iter != atomids_lookup.end(),
+              "QtObservedInstanceBoard::recieveNotification() - atom id "
+              +toString(oldatomId)+" unknown to lookup.");
+          const ObservedValue_Index_t index = iter->second;
+          atomids_lookup.erase(iter);
+          atomids_lookup.insert( std::pair<atomId_t, ObservedValue_Index_t>(newatomId, index) );
+        }
         // no need update SignedOn, ref does not change
         emit atomIndexChanged(oldatomId, newatomId);
@@ -232 +241 @@
 }
 
-void QtObservedInstanceBoard::atomcountsubjectKilled(const atomId_t _atomid)
-{
-  LOG(3, "DEBUG: InstanceBoard emits atomRemoved for " << _atomid);
-  emit atomRemoved(_atomid);
-}
-
-void QtObservedInstanceBoard::moleculecountsubjectKilled(const moleculeId_t _molid)
-{
-  LOG(3, "DEBUG: InstanceBoard emits moleculeRemoved for " << _molid);
-  emit moleculeRemoved(_molid);
+void QtObservedInstanceBoard::atomcountsubjectKilled(ObservedValue_Index_t _id)
+{
+  LOG(3, "DEBUG: InstanceBoard emits atomRemoved for " << _id);
+  emit atomRemoved(_id);
+}
+
+void QtObservedInstanceBoard::moleculecountsubjectKilled(ObservedValue_Index_t _id)
+{
+  LOG(3, "DEBUG: InstanceBoard emits moleculeRemoved for " << _id);
+  emit moleculeRemoved(_id);
 }
 
 QtObservedAtom::ptr QtObservedInstanceBoard::getObservedAtom(const atomId_t _id)
 {
-  return atomObservedValues.get(_id);
-}
-
-QtObservedAtom::ptr QtObservedInstanceBoard::getObservedAtom(const Observable * _obs)
-{
-  const atom * walker = dynamic_cast<const atom *>(_obs);
-  return atomObservedValues.get(walker->getId());
+  return atomObservedValues.get(atomids_lookup[_id]);
 }
 
 QtObservedMolecule::ptr QtObservedInstanceBoard::getObservedMolecule(const moleculeId_t _id)
 {
-  return moleculeObservedValues.get(_id);
-}
-
-QtObservedMolecule::ptr QtObservedInstanceBoard::getObservedMolecule(const Observable * _obs)
-{
-  const molecule * mol = dynamic_cast<const molecule *>(_obs);
-  return moleculeObservedValues.get(mol->getId());
-}
-
-void QtObservedInstanceBoard::markObservedAtomAsConnected(const atomId_t _id)
+  return moleculeObservedValues.get(moleculeids_lookup[_id]);
+}
+
+void QtObservedInstanceBoard::markObservedAtomAsConnected(ObservedValue_Index_t _id)
 {
   atomObservedValues.markObservedValuesAsConnected(_id);
 }
 
-void QtObservedInstanceBoard::markObservedAtomAsDisconnected(const atomId_t _id)
+void QtObservedInstanceBoard::markObservedAtomAsDisconnected(ObservedValue_Index_t _id)
 {
   atomObservedValues.markObservedValuesAsDisconnected(_id);
 }
 
-void QtObservedInstanceBoard::markObservedAtomForErase(const atomId_t _id)
+void QtObservedInstanceBoard::markObservedAtomForErase(ObservedValue_Index_t _id)
 {
   atomObservedValues.eraseObservedValues(_id);
 }
 
-void QtObservedInstanceBoard::markObservedMoleculeAsConnected(const moleculeId_t _id)
+void QtObservedInstanceBoard::markObservedMoleculeAsConnected(ObservedValue_Index_t _id)
 {
   moleculeObservedValues.markObservedValuesAsConnected(_id);
 }
 
-void QtObservedInstanceBoard::markObservedMoleculeAsDisconnected(const moleculeId_t _id)
+void QtObservedInstanceBoard::markObservedMoleculeAsDisconnected(ObservedValue_Index_t _id)
 {
   moleculeObservedValues.markObservedValuesAsDisconnected(_id);
 }
 
-void QtObservedInstanceBoard::markObservedMoleculeForErase(const moleculeId_t _id)
+void QtObservedInstanceBoard::markObservedMoleculeForErase(ObservedValue_Index_t _id)
 {
   moleculeObservedValues.eraseObservedValues(_id);