--- trunk/src/math/ParallelRandNumGen.cpp	2005/04/15 22:04:00	507
+++ branches/development/src/math/ParallelRandNumGen.cpp	2010/07/09 23:08:25	1465
@@ -6,19 +6,10 @@
  * redistribute this software in source and binary code form, provided
  * that the following conditions are met:
  *
- * 1. Acknowledgement of the program authors must be made in any
- *    publication of scientific results based in part on use of the
- *    program.  An acceptable form of acknowledgement is citation of
- *    the article in which the program was described (Matthew
- *    A. Meineke, Charles F. Vardeman II, Teng Lin, Christopher
- *    J. Fennell and J. Daniel Gezelter, "OOPSE: An Object-Oriented
- *    Parallel Simulation Engine for Molecular Dynamics,"
- *    J. Comput. Chem. 26, pp. 252-271 (2005))
- *
- * 2. Redistributions of source code must retain the above copyright
+ * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  *
- * 3. Redistributions in binary form must reproduce the above copyright
+ * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the
  *    distribution.
@@ -37,23 +28,34 @@
  * arising out of the use of or inability to use software, even if the
  * University of Notre Dame has been advised of the possibility of
  * such damages.
+ *
+ * SUPPORT OPEN SCIENCE!  If you use OpenMD or its source code in your
+ * research, please cite the appropriate papers when you publish your
+ * work.  Good starting points are:
+ *                                                                      
+ * [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).          
+ * [4]  Vardeman & Gezelter, in progress (2009).                        
  */
 
 #include "math/ParallelRandNumGen.hpp"
 #ifdef IS_MPI
 #include <mpi.h>
+#endif
 
-namespace oopse {
+namespace OpenMD {
 
-
-
   int ParallelRandNumGen::nCreatedRNG_ = 0;
 
-  ParallelRandNumGen::ParallelRandNumGen( const uint32& oneSeed) {
+  ParallelRandNumGen::ParallelRandNumGen(const uint32& oneSeed) {
 
     const int masterNode = 0;
-    int seed = oneSeed;
+    unsigned long seed = oneSeed;
+
+#ifdef IS_MPI
     MPI_Bcast(&seed, 1, MPI_UNSIGNED_LONG, masterNode, MPI_COMM_WORLD); 
+#endif
 
     if (seed != oneSeed) {
       sprintf(painCave.errMsg,
@@ -63,14 +65,20 @@ namespace oopse {
     }
 
     int nProcessors;
-    MPI_Comm_size(MPI_COMM_WORLD, &nProcessors);
+#ifdef IS_MPI
+    MPI_Comm_size( MPI_COMM_WORLD, &nProcessors);
     MPI_Comm_rank( MPI_COMM_WORLD, &myRank_);
-    //In order to generate independent random number stream, the actual seed used by random 
-    //number generator is the seed passed to the constructor plus the number of random number
-    //generators which are already created.     
-    int newSeed = oneSeed + nCreatedRNG_;
+#else
+    nProcessors = 1;
+    myRank_ = 0;
+#endif
+    //In order to generate independent random number stream, the
+    //actual seed used by random number generator is the seed passed
+    //to the constructor plus the number of random number generators
+    //which are already created.
+    unsigned long newSeed = oneSeed + nCreatedRNG_;
     mtRand_ = new MTRand(newSeed, nProcessors, myRank_);
-
+    
     ++nCreatedRNG_;
   }
 
@@ -79,30 +87,37 @@ namespace oopse {
     std::vector<uint32> bigSeed;
     const int masterNode = 0;
     int nProcessors;
-    MPI_Comm_size(MPI_COMM_WORLD, &nProcessors);
+#ifdef IS_MPI
+    MPI_Comm_size( MPI_COMM_WORLD, &nProcessors);
     MPI_Comm_rank( MPI_COMM_WORLD, &myRank_);
+#else
+    nProcessors = 1;
+    myRank_ = 0;
+#endif
     mtRand_ = new MTRand(nProcessors, myRank_);
 
-    seed();       /** @todo calling virtual function in constructor is not a good design */
+    seed();       /** @todo calling virtual function in constructor is
+                      not a good design */
   }
 
 
   void ParallelRandNumGen::seed( const uint32 oneSeed ) {
 
     const int masterNode = 0;
-    int seed = oneSeed;
+    unsigned long seed = oneSeed;
+#ifdef IS_MPI
     MPI_Bcast(&seed, 1, MPI_UNSIGNED_LONG, masterNode, MPI_COMM_WORLD); 
-
+#endif
     if (seed != oneSeed) {
       sprintf(painCave.errMsg,
 	      "Using different seed to initialize ParallelRandNumGen.\n");
       painCave.isFatal = 1;;
       simError();
     }
-
-    int newSeed = oneSeed +nCreatedRNG_;
+    
+    unsigned long newSeed = oneSeed +nCreatedRNG_;
     mtRand_->seed(newSeed);
-
+    
     ++nCreatedRNG_;
   }
         
@@ -111,17 +126,22 @@ namespace oopse {
     std::vector<uint32> bigSeed;
     int size;
     const int masterNode = 0;
+#ifdef IS_MPI
     if (worldRank == masterNode) {
+#endif
+
       bigSeed = mtRand_->generateSeeds();
       size = bigSeed.size();
+
+#ifdef IS_MPI
       MPI_Bcast(&size, 1, MPI_INT, masterNode, MPI_COMM_WORLD);        
       MPI_Bcast(&bigSeed[0], size, MPI_UNSIGNED_LONG, masterNode, MPI_COMM_WORLD); 
-
     }else {
       MPI_Bcast(&size, 1, MPI_INT, masterNode, MPI_COMM_WORLD);        
       bigSeed.resize(size);
       MPI_Bcast(&bigSeed[0], size, MPI_UNSIGNED_LONG, masterNode, MPI_COMM_WORLD); 
     }
+#endif
     
     if (bigSeed.size() == 1) {
       mtRand_->seed(bigSeed[0]);
@@ -130,9 +150,5 @@ namespace oopse {
     }
 
     ++nCreatedRNG_;
-  }    
-
-
+  }        
 }
-
-#endif