Clean up
authorNikhil Jain <nikhil@illinois.edu>
Tue, 25 Oct 2011 02:41:13 +0000 (21:41 -0500)
committerNikhil Jain <nikhil@illinois.edu>
Tue, 25 Oct 2011 02:41:13 +0000 (21:41 -0500)
ActiveBinInfo.h
DataManager.h
Descriptor.h
Main.cpp
Node.h
Parameters.h

index 81f45dd..1faee61 100644 (file)
@@ -36,7 +36,6 @@ struct ActiveBinInfo{
 
     Key kfirst, klast;
 
-    //CkPrintf("[%d] add new node %ld\n", CkMyPe(), node->getKey());
     newvec->push_back(pr);
     int np = node->getNumParticles();
     Particle *particles = node->getParticles();
@@ -55,7 +54,6 @@ struct ActiveBinInfo{
       (*oldvec)[bin].second = true;
       Node<T> *node = (*oldvec)[bin].first;
 
-      //CkPrintf("[%d] processRefine node %ld\n", CkMyPe(), node->getKey());
       refine(node);
 
       Key kfirst, klast;
@@ -66,7 +64,6 @@ struct ActiveBinInfo{
         Node<T> *child = node->getChild(i);
         pr.first = child;
         
-        //CkPrintf("[%d] add child node %ld\n", CkMyPe(), child->getKey());
         newvec->push_back(pr);
         int np = child->getNumParticles();
         Particle *particles = child->getParticles();
@@ -90,7 +87,6 @@ struct ActiveBinInfo{
     for(int i = 0; i < nEmptyBins; i++){
       int bin = emptyBins[i];
       Node<T> *node = (*oldvec)[bin].first;
-      //node->core.type = NodeType::Empty;
     }
   }
 
@@ -151,21 +147,10 @@ struct OwnershipActiveBinInfo : public ActiveBinInfo<T> {
     int start = (ownerStart<<1);
     int end = (ownerEnd<<1)+2;
 
-#ifdef VERBOSE_NODE_REFINE
-    CkPrintf("(%d) node %lu considering tps %d (%d) - %d (%d)\n", 
-                CkMyPe(), node->getKey(), node->getOwnerStart(), start, node->getOwnerEnd(), end);
-    for(int i = node->getOwnerStart(); i <= node->getOwnerEnd(); i++){
-      CkPrintf("(%d) tp %d key %lx - %lx\n",CkMyPe(),i,owners[(i<<1)],owners[(i<<1)+1]);
-    }
-#endif
-    
     Node<T> *children = node->getChildren();
     Key childKey = children[0].getKey();
 
     children[0].setOwnerStart(ownerStart);
-#ifdef VERBOSE_NODE_REFINE
-    CkPrintf("(%d) child %d key %lu ownerStart %d\n", CkMyPe(), 0, childKey, node->getOwnerStart());
-#endif
     int childDepth = children[0].getDepth();
     childKey++;
 
@@ -178,32 +163,18 @@ struct OwnershipActiveBinInfo : public ActiveBinInfo<T> {
       // array (a lo and a hi), we divide by two to get the tp
       int tp_idx = (owner_idx>>1);
 
-#ifdef VERBOSE_NODE_REFINE
-      CkPrintf("(%d) check tps for child %d key %lu : idx %d tp %d\n",
-                CkMyPe(), i, testKey, owner_idx, tp_idx);
-#endif
-
       // is the range of the tree piece contained completely
       // within the child node or does it straddle this child
       // and the previous one?
       if(EVEN(owner_idx)) children[i-1].setOwnerEnd(tp_idx-1);
       else                children[i-1].setOwnerEnd(tp_idx);
 
-#ifdef VERBOSE_NODE_REFINE
-      CkPrintf("(%d) child %d key %lu ownerend %d\n", CkMyPe(), i-1, children[i-1].getKey(), children[i-1].getOwnerEnd());
-#endif
       if(children[i-1].getOwnerEnd() < children[i-1].getOwnerStart()){
         children[i-1].setOwners(-69,-69);
         children[i-1].setType(EmptyBucket);
-#ifdef VERBOSE_NODE_REFINE
-        CkPrintf("(%d) child %d key %lu EMPTYBUCKET\n", CkMyPe(), i-1, children[i-1].getKey());
-#endif
       }
 
       children[i].setOwnerStart(tp_idx);
-#ifdef VERBOSE_NODE_REFINE
-      CkPrintf("(%d) child %d key %lu ownerstart %d\n", CkMyPe(), i, children[i].getKey(), children[i].getOwnerStart());
-#endif
 
       start = owner_idx;
 
@@ -211,13 +182,7 @@ struct OwnershipActiveBinInfo : public ActiveBinInfo<T> {
     }
 
     children[BRANCH_FACTOR-1].setOwnerEnd(ownerEnd);
-#ifdef VERBOSE_NODE_REFINE
-    CkPrintf("(%d) child %d key %lu ownerend %d\n", CkMyPe(), BRANCH_FACTOR-1, children[BRANCH_FACTOR-1].getKey(), children[BRANCH_FACTOR-1].getOwnerEnd());
-#endif
     if(children[BRANCH_FACTOR-1].getOwnerEnd() < children[BRANCH_FACTOR-1].getOwnerStart()){
-#ifdef VERBOSE_NODE_REFINE
-      CkPrintf("(%d) child %d key %lu EMPTYBUCKET\n", CkMyPe(), BRANCH_FACTOR-1, children[BRANCH_FACTOR-1].getKey());
-#endif
       children[BRANCH_FACTOR-1].setOwners(-171,-171);
       children[BRANCH_FACTOR-1].setType(EmptyBucket);
     }
index 6973a20..60b1380 100644 (file)
@@ -170,7 +170,6 @@ class DataManager : public CBase_DataManager {
   
   // called by tree pieces
   void submitParticles(CkVec<ParticleMsg *> *vec, int numParticles, TreePiece *tp, Key smallestKey, Key largestKey); 
-  //void requestMoments(MomentsRequestMsg *msg);
   void requestMoments(Key k, int replyTo);
   void advance(CkReductionMsg *);
 #ifdef STATISTICS
index 255c951..df2aa50 100644 (file)
@@ -71,30 +71,8 @@ struct NodeDescriptor {
   {
   }
 
-  /*
-  NodeDescriptor(Key nk) :
-    numParticles(0), 
-    nodeKey(nk) 
-  {
-    largestKey = Key(0);
-    smallestKey = ~largestKey;
-  }
-  */
-
-  void grow(const NodeDescriptor &other){
-    /*
-    CkPrintf("(%d) grow1 %d sk %lx lk %lx grow2 %d sk %lx lk %lx\n",
-              CkMyPe(),
-              numParticles, 
-              smallestKey, 
-              largestKey,
-              other.numParticles, 
-              other.smallestKey, 
-              other.largestKey
-              );
-    */
-    if(other.numParticles == 0){
-      //CkPrintf("(%d) growresult 1\n", CkMyPe());
+ void grow(const NodeDescriptor &other){
+   if(other.numParticles == 0){
       return;
     }
 
@@ -102,12 +80,10 @@ struct NodeDescriptor {
       smallestKey = other.smallestKey;
       largestKey = other.largestKey;
       numParticles = other.numParticles;
-      //CkPrintf("(%d) growresult 2\n", CkMyPe());
     } else {
       if(smallestKey > other.smallestKey) smallestKey = other.smallestKey;
       if(largestKey < other.largestKey) largestKey = other.largestKey;
       numParticles += other.numParticles;
-      //CkPrintf("(%d) growresult 3\n", CkMyPe());
     }
 
   }
index 49c85cc..dd679fa 100644 (file)
--- a/Main.cpp
+++ b/Main.cpp
@@ -24,17 +24,8 @@ Main::Main(CkArgMsg *msg){
   dataManagerProxy = CProxy_DataManager::ckNew();
 
   CkArrayOptions opts(globalParams.numTreePieces);
-  /*
-  CProxy_RRMap myMap = CProxy_RRMap::ckNew();
-  opts.setMap(myMap);
-  */
   treePieceProxy = CProxy_TreePiece::ckNew(opts);
 
-#ifdef TRACE_REMOTE_DATA_REQUESTS
-  traceRegisterUserEvent("Node",REMOTE_NODE_REQUEST);
-  traceRegisterUserEvent("Particles",REMOTE_PARTICLE_REQUEST);
-#endif
-
   mainProxy = thisProxy;
 
   thisProxy.commence();
diff --git a/Node.h b/Node.h
index 80373bb..0d34190 100644 (file)
--- a/Node.h
+++ b/Node.h
@@ -73,28 +73,15 @@ class Node {
 
   public:
   T data;
-#ifdef CHECK_NUM_INTERACTIONS
-  CmiUInt8 nodeInteractions;
-  CmiUInt8 partInteractions;
-#endif
-
   Node(Key k, int d, Particle *p, int np, Node<T> *par=NULL) : 
     core(k,d,p,np), parent(par), children(NULL), data(),
-#ifdef CHECK_NUM_INTERACTIONS
-    nodeInteractions(0),
-    partInteractions(0),
-#endif
-    numChildrenMomentsReady(0)
+   numChildrenMomentsReady(0)
   {
   }
 
   Node() : 
     core(), parent(NULL), children(NULL), data(),
-#ifdef CHECK_NUM_INTERACTIONS
-    nodeInteractions(0),
-    partInteractions(0),
-#endif
-    numChildrenMomentsReady(0)
+   numChildrenMomentsReady(0)
   {
   }
 
@@ -219,7 +206,6 @@ class Node {
     int end = getNumParticles();
     Particle *particles = getParticles();
     int splitters[BRANCH_FACTOR+1];
-    //splitters.resize(BRANCH_FACTOR+1);
 
     Key childKey = (myKey << numRankBits);
     int childDepth = depth+1;
@@ -243,8 +229,6 @@ class Node {
     child->setDepth(childDepth);
     child->setParent(this);
 
-    //CkPrintf("(%d) child %lu splitteridx %d\n", CkMyPe(), child->getKey(), childPartStart);
-
     // findSplitters distributed particles over
     // different children
     child->core.particleStart = core.particleStart+childPartStart;
@@ -389,15 +373,6 @@ class Node {
 
   }
 
-#ifdef CHECK_NUM_INTERACTIONS
-  void addNodeInteractions(CmiUInt8 pn){
-    nodeInteractions += pn;
-  }
-
-  void addPartInteractions(CmiUInt8 pp){
-    partInteractions += pp;
-  }
-#endif
 };
 
 template<typename T>
@@ -407,15 +382,7 @@ ostream &operator<<(ostream &os, const Node<T> &node){
   os << node.getKey() 
      << " [label=\"" << node.getKey() 
      << " (" << node.getNumParticles() << ","
-#ifdef CHECK_NUM_INTERACTIONS
-     << node.nodeInteractions << ","
-     << node.partInteractions << ","
-#endif
-     /*
-     << "\\n" << oss.str() << ","
-     << node.data.moments.totalMass << ","
-     */
-     << "\\n" << node.data.moments.rsq << ","
+    << "\\n" << node.data.moments.rsq << ","
      << "\\n" << node.getOwnerStart() << ":" 
      << node.getOwnerEnd() << ")\\n"
      << NodeTypeString[node.getType()]
@@ -431,8 +398,6 @@ ostream &operator<<(ostream &os, const Node<T> &node){
 
   os << "]" << endl;
 
-  //if(node.getType() == Internal) return os;
-
   for(int i = 0; i < node.getNumChildren(); i++){
     Node<T> &childref = *(node.getChild(i));
     os << node.getKey() << " -> " << childref.getKey() << endl;
index a5d2694..d862619 100644 (file)
@@ -42,7 +42,6 @@ struct Parameters {
     p | dthf;
     p | tolsq;
     p | epssq;
-    //p | branchFactor;
     p | ppc;
     p | ppb;
     p | yieldPeriod;
@@ -70,7 +69,6 @@ struct Parameters {
     if(it != table.end()){
       return it->second;
     }
-    //cerr << "getparam: " << name.c_str() << " unknown" << endl;
     return string();
   }