From 851e176590514bd2a0e306d0b5896f4598fff85d Mon Sep 17 00:00:00 2001
From: Seokju Chung <seokju.chung@columbia.edu>
Date: Tue, 21 Apr 2026 09:07:23 -0500
Subject: [PATCH] Return false instead of exit

---
 sbncode/SinglePhotonAnalysis/NCDeltaRadiative_module.cc    | 4 ++--
 sbncode/SinglePhotonAnalysis/NCRadiativeResonant_module.cc | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/sbncode/SinglePhotonAnalysis/NCDeltaRadiative_module.cc b/sbncode/SinglePhotonAnalysis/NCDeltaRadiative_module.cc
index aff90eb39..892d31f06 100644
--- a/sbncode/SinglePhotonAnalysis/NCDeltaRadiative_module.cc
+++ b/sbncode/SinglePhotonAnalysis/NCDeltaRadiative_module.cc
@@ -161,7 +161,7 @@ bool NCDeltaRadiative::filter(art::Event & e) {
       simb::MCParticle const & mcp = mct.GetParticle(i);
       if(mcp.TrackId() != i) {
   std::cout << "ERROR: " << __LINE__ << " " << __PRETTY_FUNCTION__ << "\nTrackId does not match index\n";
-  exit(1);
+  return false;
       }
       if(!(mcp.StatusCode() == 1 && mcp.PdgCode() == 22)) continue;
       exiting_photon_parents.push_back(mcp.Mother());
@@ -173,7 +173,7 @@ bool NCDeltaRadiative::filter(art::Event & e) {
     //CHEKC hardcode, TPC filter:
     if(abs(mcp.Vx())>210 ||  abs(mcp.Vy())>210||mcp.Vz()>510 || mcp.Vz()<-1){
       std::cout<<"OUTSIDE TPC x y z ="<<mcp.Vx()<<" "<<mcp.Vy()<<" "<<mcp.Vz()<<std::endl;
-      exit(0);
+      return false;
     }
     if(abs(mcp.PdgCode()) == 2114 || abs(mcp.PdgCode()) == 2214) {
   if(ftree) FillTree(e, i, mcp.PdgCode(), true);
diff --git a/sbncode/SinglePhotonAnalysis/NCRadiativeResonant_module.cc b/sbncode/SinglePhotonAnalysis/NCRadiativeResonant_module.cc
index 9e46ade60..5665c8bfa 100644
--- a/sbncode/SinglePhotonAnalysis/NCRadiativeResonant_module.cc
+++ b/sbncode/SinglePhotonAnalysis/NCRadiativeResonant_module.cc
@@ -185,12 +185,12 @@ bool NCRadiativeResonant::filter(art::Event & e) {
       //CHEKC hardcode, TPC filter:
       if(abs(mcp.Vx())>210 ||  abs(mcp.Vy())>210||mcp.Vz()>510 || mcp.Vz()<-1){
       std::cout<<"OUTSIDE TPC x y z ="<<mcp.Vx()<<" "<<mcp.Vy()<<" "<<mcp.Vz()<<std::endl;
-      exit(0);
+      return false;
       }
 
       if(mcp.TrackId() != jth) {
         std::cout << "ERROR: " << __LINE__ << " " << __PRETTY_FUNCTION__ << "\nTrackId does not match index\n";
-        exit(1);
+        return false;
       }
       if(!(mcp.StatusCode() == 1 && mcp.PdgCode() == 22)) continue;//next, if this is not a photon with StatusCode 1
       exiting_photon_parents.push_back(mcp.Mother());//collect photon's parents