From df9999fe1903d7d361dee533bbadacbab06c6a05 Mon Sep 17 00:00:00 2001
From: Daiki Sekihata <daiki.sekihata@cern.ch>
Date: Wed, 11 Mar 2026 19:19:03 +0100
Subject: [PATCH] PWGEM/Dilepton: update skimmerPrimaryElectronQC.cxx

---
 .../TableProducer/skimmerPrimaryElectronQC.cxx    | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/PWGEM/Dilepton/TableProducer/skimmerPrimaryElectronQC.cxx b/PWGEM/Dilepton/TableProducer/skimmerPrimaryElectronQC.cxx
index 335fb978425..b50a0df8bab 100644
--- a/PWGEM/Dilepton/TableProducer/skimmerPrimaryElectronQC.cxx
+++ b/PWGEM/Dilepton/TableProducer/skimmerPrimaryElectronQC.cxx
@@ -75,13 +75,14 @@ struct skimmerPrimaryElectronQC {
   struct : ConfigurableGroup {
     std::string prefix = "trackcut";
     Configurable<int> min_ncluster_tpc{"min_ncluster_tpc", 0, "min ncluster tpc"};
-    Configurable<int> mincrossedrows{"mincrossedrows", 40, "min. crossed rows"};
+    Configurable<int> mincrossedrows{"mincrossedrows", 0, "min. crossed rows"};
     Configurable<int> min_ncluster_its{"min_ncluster_its", 2, "min ncluster its"};
     Configurable<int> min_ncluster_itsib{"min_ncluster_itsib", 0, "min ncluster itsib"};
-    Configurable<float> min_tpc_cr_findable_ratio{"min_tpc_cr_findable_ratio", 0.8, "min. TPC Ncr/Nf ratio"};
+    Configurable<float> min_tpc_cr_findable_ratio{"min_tpc_cr_findable_ratio", 0.0, "min. TPC Ncr/Nf ratio"};
     Configurable<float> max_frac_shared_clusters_tpc{"max_frac_shared_clusters_tpc", 999.f, "max fraction of shared clusters in TPC"};
-    Configurable<float> maxchi2tpc{"maxchi2tpc", 5.0, "max. chi2/NclsTPC"};
-    Configurable<float> maxchi2its{"maxchi2its", 36.0, "max. chi2/NclsITS"};
+    Configurable<float> maxchi2tpc{"maxchi2tpc", 1e+10, "max. chi2/NclsTPC"};
+    Configurable<float> minchi2tpc{"minchi2tpc", -1e+10, "min. chi2/NclsTPC"};
+    Configurable<float> maxchi2its{"maxchi2its", 1e+10, "max. chi2/NclsITS"}; // actual maximum is 36 in the reconstruction.
     Configurable<float> minchi2its{"minchi2its", -1e+10, "min. chi2/NclsITS"};
     Configurable<float> minpt{"minpt", 0.05, "min pt for ITS-TPC track"};
     Configurable<float> maxeta{"maxeta", 0.9, "eta acceptance"};
@@ -94,9 +95,9 @@ struct skimmerPrimaryElectronQC {
 
   struct : ConfigurableGroup {
     std::string prefix = "tighttrackcut";
-    Configurable<int> min_ncluster_tpc_pid{"min_ncluster_tpc_pid", 60, "min ncluster tpc used for PID"};
+    Configurable<int> min_ncluster_tpc_pid{"min_ncluster_tpc_pid", 0, "min ncluster tpc used for PID"};
     Configurable<int> min_ncluster_tpc{"min_ncluster_tpc", 0, "min ncluster tpc"};
-    Configurable<int> mincrossedrows{"mincrossedrows", 100, "min. crossed rows"};
+    Configurable<int> mincrossedrows{"mincrossedrows", 120, "min. crossed rows"};
     Configurable<int> min_ncluster_its{"min_ncluster_its", 5, "min ncluster its"};
     Configurable<int> min_ncluster_itsib{"min_ncluster_itsib", 3, "min ncluster itsib"};
     Configurable<float> min_tpc_cr_findable_ratio{"min_tpc_cr_findable_ratio", 0.8, "min. TPC Ncr/Nf ratio"};
@@ -309,7 +310,7 @@ struct skimmerPrimaryElectronQC {
     }
 
     if (track.hasTPC()) {
-      if (track.tpcChi2NCl() < 0.f || trackcut.maxchi2tpc < track.tpcChi2NCl()) {
+      if (track.tpcChi2NCl() < trackcut.minchi2tpc || trackcut.maxchi2tpc < track.tpcChi2NCl()) {
         return false;
       }