Changeset 41a467 for src/Actions

Timestamp:

Nov 3, 2011, 7:44:01 PM (13 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:

ab26c3

Parents:

47d041

git-author:

Frederik Heber <heber@…> (10/27/11 15:08:36)

git-committer:

Frederik Heber <heber@…> (11/03/11 19:44:01)

Message:

LARGE: config class is now just a tiny container.

• this was loooooobg overdue. Config.cpp contained remnants from parsing pcp files and much else. Also fragmentation depended on it. Since refactoring of MoleculeListClass and the fragmentation, we don't need it anymore.
• helper functions ParseForParameters(), LoadMolecule() extracted into new module PcpParser_helper.
• config class now just contains 4 variables that are generally required (especially IsAngstroem) and they should probably remain with the world.
• removed some places where config.hpp was no unnecessarily included.
• Moved ConfigFileBuffer.* over to subfolder src/Parser/ where it rather belongs (associated with PcpParser).

Location:

src/Actions

Files:

• CommandAction/BondLengthTableAction.cpp (modified) (1 diff)
• FragmentationAction/FragmentationAction.cpp (modified) (1 diff)
• GraphAction/SubgraphDissectionAction.cpp (modified) (1 diff)
• MoleculeAction/FillVoidWithMoleculeAction.cpp (modified) (1 diff)
• MoleculeAction/FillWithMoleculeAction.cpp (modified) (1 diff)
• MoleculeAction/SaveAdjacencyAction.cpp (modified) (1 diff)
• MoleculeAction/SaveBondsAction.cpp (modified) (1 diff)
• MoleculeAction/SuspendInWaterAction.cpp (modified) (2 diffs)

src/Actions/CommandAction/BondLengthTableAction.cpp

@@ -24 +24 @@
 #include "CodePatterns/MemDebug.hpp"
 
-#include "Graph/BondGraph.hpp"
-#include "config.hpp"
 #include "CodePatterns/Log.hpp"
 #include "CodePatterns/Verbose.hpp"
+#include "Graph/BondGraph.hpp"
 #include "World.hpp"
 

src/Actions/FragmentationAction/FragmentationAction.cpp

@@ -21 +21 @@
 
 #include "atom.hpp"
-#include "Graph/BondGraph.hpp"
-#include "config.hpp"
 #include "CodePatterns/Log.hpp"
-#include "CodePatterns/Verbose.hpp"
 #include "Fragmentation/Fragmentation.hpp"
 #include "Fragmentation/HydrogenSaturation_enum.hpp"
 #include "Graph/DepthFirstSearchAnalysis.hpp"
 #include "molecule.hpp"
-#include "Descriptors/MoleculeDescriptor.hpp"
 #include "World.hpp"
 

src/Actions/GraphAction/SubgraphDissectionAction.cpp

@@ -25 +25 @@
 #include "atom.hpp"
 #include "Bond/bond.hpp"
-#include "config.hpp"
 #include "CodePatterns/Log.hpp"
 #include "CodePatterns/Verbose.hpp"

src/Actions/MoleculeAction/FillVoidWithMoleculeAction.cpp

@@ -21 +21 @@
 
 #include "atom.hpp"
-#include "Graph/BondGraph.hpp"
-#include "Tesselation/boundary.hpp"
 #include "CodePatterns/Verbose.hpp"
-#include "config.hpp"
 #include "Descriptors/MoleculeIdDescriptor.hpp"
 #include "Descriptors/MoleculeOrderDescriptor.hpp"
+#include "Graph/BondGraph.hpp"
 #include "molecule.hpp"
 #include "MoleculeListClass.hpp"
 #include "Parser/FormatParserInterface.hpp"
 #include "Parser/FormatParserStorage.hpp"
+#include "Tesselation/boundary.hpp"
 #include "World.hpp"
 

src/Actions/MoleculeAction/FillWithMoleculeAction.cpp

@@ -21 +21 @@
 
 #include "atom.hpp"
-#include "Graph/BondGraph.hpp"
-#include "Tesselation/boundary.hpp"
 #include "CodePatterns/Verbose.hpp"
-#include "config.hpp"
 #include "Descriptors/MoleculeIdDescriptor.hpp"
 #include "Descriptors/MoleculeOrderDescriptor.hpp"
+#include "Graph/BondGraph.hpp"
 #include "molecule.hpp"
 #include "MoleculeListClass.hpp"
 #include "Parser/FormatParserInterface.hpp"
 #include "Parser/FormatParserStorage.hpp"
+#include "Tesselation/boundary.hpp"
 #include "World.hpp"
 

src/Actions/MoleculeAction/SaveAdjacencyAction.cpp

@@ -20 +20 @@
 #include "CodePatterns/MemDebug.hpp"
 
-#include "Graph/BondGraph.hpp"
-#include "config.hpp"
-#include "CodePatterns/Log.hpp"
-#include "molecule.hpp"
-#include "CodePatterns/Verbose.hpp"
-#include "World.hpp"
-
 #include <iostream>
 #include <fstream>
 #include <string>
+
+#include "CodePatterns/Log.hpp"
+#include "Graph/BondGraph.hpp"
+#include "molecule.hpp"
+#include "World.hpp"
 
 #include "Actions/MoleculeAction/SaveAdjacencyAction.hpp"

src/Actions/MoleculeAction/SaveBondsAction.cpp

@@ -20 +20 @@
 #include "CodePatterns/MemDebug.hpp"
 
-#include "Graph/BondGraph.hpp"
-#include "config.hpp"
-#include "CodePatterns/Log.hpp"
-#include "molecule.hpp"
-#include "CodePatterns/Verbose.hpp"
-#include "World.hpp"
-
 #include <iostream>
 #include <fstream>
 #include <string>
+
+#include "CodePatterns/Log.hpp"
+#include "Graph/BondGraph.hpp"
+#include "molecule.hpp"
+#include "World.hpp"
 
 #include "Actions/MoleculeAction/SaveBondsAction.hpp"

src/Actions/MoleculeAction/SuspendInWaterAction.cpp

@@ -20 +20 @@
 #include "CodePatterns/MemDebug.hpp"
 
+#include "CodePatterns/Log.hpp"
 #include "Tesselation/boundary.hpp"
-#include "config.hpp"
-#include "CodePatterns/Log.hpp"
-#include "CodePatterns/Verbose.hpp"
 #include "World.hpp"
 
@@ -30 +28 @@
 
 #include "Actions/MoleculeAction/SuspendInWaterAction.hpp"
+
+class config;
 
 using namespace MoleCuilder;