--- branches/development/src/flucq/FluctuatingChargePropagator.cpp	2012/06/06 19:54:09	1748
+++ trunk/src/flucq/FluctuatingChargePropagator.cpp	2013/07/19 21:25:45	1908
@@ -35,7 +35,7 @@
  *                                                                      
  * [1]  Meineke, et al., J. Comp. Chem. 26, 252-271 (2005).             
  * [2]  Fennell & Gezelter, J. Chem. Phys. 124, 234104 (2006).          
- * [3]  Sun, Lin & Gezelter, J. Chem. Phys. 128, 24107 (2008).          
+ * [3]  Sun, Lin & Gezelter, J. Chem. Phys. 128, 234107 (2008).          
  * [4]  Kuang & Gezelter,  J. Chem. Phys. 133, 164101 (2010).
  * [5]  Vardeman, Stocker & Gezelter, J. Chem. Theory Comput. 7, 834 (2011).
  */
@@ -48,8 +48,6 @@
 #include "optimization/StatusFunction.hpp"
 #include "optimization/OptimizationFactory.hpp"
 
-
-
 #ifdef IS_MPI
 #include <mpi.h>
 #endif
@@ -57,23 +55,31 @@ namespace OpenMD {
 using namespace QuantLib;
 namespace OpenMD {
 
-  FluctuatingChargePropagator::FluctuatingChargePropagator(SimInfo* info, 
-                                                           ForceManager* fm) : 
-    info_(info), forceMan_(fm), hasFlucQ_(false) {
+  FluctuatingChargePropagator::FluctuatingChargePropagator(SimInfo* info) : 
+    info_(info), hasFlucQ_(false), forceMan_(NULL) {
     
+    Globals* simParams = info_->getSimParams();
+    fqParams_ = simParams->getFluctuatingChargeParameters();    
+    fqConstraints_ = new FluctuatingChargeConstraints(info_);
+    fqConstraints_->setConstrainRegions( fqParams_->getConstrainRegions() );
+  }
+
+  FluctuatingChargePropagator::~FluctuatingChargePropagator() {
+    if (fqConstraints_ != NULL) delete fqConstraints_;
+  }
+
+  void FluctuatingChargePropagator::setForceManager(ForceManager* forceMan) {
+    forceMan_ = forceMan;
+  }
+
+  void FluctuatingChargePropagator::initialize() {
+
     if (info_->usesFluctuatingCharges()) {
       if (info_->getNFluctuatingCharges() > 0) {
-        
         hasFlucQ_ = true;
-        Globals* simParams = info_->getSimParams();
-        fqParams_ = simParams->getFluctuatingChargeParameters();
-        
       }
     }
-  }
 
-  void FluctuatingChargePropagator::initialize() {
-
     if (!hasFlucQ_) return;
 
     SimInfo::MoleculeIterator i;
@@ -89,24 +95,26 @@ namespace OpenMD {
         atom->setFlucQVel(0.0);
       }
     }
-
-    std::cerr << "doing a minimization\n";
     
-    fqConstraints_ = new FluctuatingChargeConstraints(info_);
-    FluctuatingChargeObjectiveFunction flucQobjf(info_, forceMan_, fqConstraints_);    
+    FluctuatingChargeObjectiveFunction flucQobjf(info_, forceMan_, 
+                                                 fqConstraints_);
+
     DynamicVector<RealType> initCoords = flucQobjf.setInitialCoords();
-    Problem problem(flucQobjf, *(new NoConstraint()), *(new NoStatus()), initCoords);
+    Problem problem(flucQobjf, *(new NoConstraint()), *(new NoStatus()), 
+                    initCoords);
+
     EndCriteria endCriteria(1000, 100, 1e-5, 1e-5, 1e-5);       
+
     OptimizationMethod* minim = OptimizationFactory::getInstance()->createOptimization("SD", info_);
 
-    minim->minimize(problem, endCriteria);
+    DumpStatusFunction dsf(info_);  // we want a dump file written
+                                    // every iteration
 
+    minim->minimize(problem, endCriteria);
   }
 
-
   void FluctuatingChargePropagator::applyConstraints() {
     if (!hasFlucQ_) return;
-
     fqConstraints_->applyConstraints();
   }
 }