Do not add event time for unsmeared hits (#331)

Unsmeared hits are relative to MC global time. Adding in the event time was causing an artificial shift in the hit times if smeared hit times was turned off.
ANNIEsoft · Jan 13, 2025 · 5b3a549 · 5b3a549
1 parent 4a88c36
commit 5b3a549
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/UserTools/LoadWCSim/LoadWCSim.cpp b/UserTools/LoadWCSim/LoadWCSim.cpp
@@ -658,10 +658,13 @@ bool LoadWCSim::Execute(){
 			if(verbosity>2) cout<<"digit Q is "<<digiq<<endl;
 			// Get hit parent information
 			std::vector<int> parents = GetHitParentIds(digihit, firsttrigt);
-			//std::cout <<"digittime before adding event time: "<<digittime<<","<<EventTimeNs<<std::endl;
+			if(verbosity>2){ std::cout <<"digittime before adding event time: "<<digittime<<","<<EventTimeNs<<std::endl; }
 
-			if (!splitSubtriggers) digittime += EventTimeNs;			
-			//std::cout <<"digittime after adding event time: "<<digittime<<std::endl;
+			// Unsmeared hit times are with respect to the MC global time --> therefore we shouldn't add the event / trigger time if using unsmeared
+			if(use_smeared_digit_time){
+				if (!splitSubtriggers) digittime += EventTimeNs;
+			}			
+			if(verbosity>2){ std::cout <<"digittime after adding event time: "<<digittime<<std::endl; }
 
 			MCHit nexthit(key, digittime, digiq, parents);
 			if(MCHits->count(key)==0) MCHits->emplace(key, std::vector<MCHit>{nexthit});