Make PV reader verbosity levels more selective

shahor02 · shahor02 · commit 4ee378d03312 · 2023-03-22T00:23:42.000+01:00
diff --git a/Detectors/GlobalTrackingWorkflow/readers/src/PrimaryVertexReaderSpec.cxx b/Detectors/GlobalTrackingWorkflow/readers/src/PrimaryVertexReaderSpec.cxx
@@ -116,17 +116,21 @@ void PrimaryVertexReader::run(ProcessingContext& pc)
             nambig++;
           }
         }
-        LOGP(info, "{} : total attached: {}, contributors: {}, ambiguous: {}", GIndex::getSourceName(is), mPV2MatchIdxRef[cnt].getEntriesOfSource(is), ncontrib, nambig);
+        if (mPV2MatchIdxRef[cnt].getEntriesOfSource(is)) {
+          LOGP(info, "{} : total attached: {}, contributors: {}, ambiguous: {}", GIndex::getSourceName(is), mPV2MatchIdxRef[cnt].getEntriesOfSource(is), ncontrib, nambig);
+        }
         if (mVerbose < 2) {
           continue;
         }
         std::string trIDs;
         int cntT = 0;
         for (int i = idMin; i < idMax; i++) {
-          trIDs += mPV2MatchIdx[i].asString() + " ";
-          if (!((++cntT) % 15)) {
-            LOG(info) << trIDs;
-            trIDs = "";
+          if (mVerbose > 2 || mPV2MatchIdx[i].isPVContributor()) {
+            trIDs += mPV2MatchIdx[i].asString() + " ";
+            if (!((++cntT) % 15)) {
+              LOG(info) << trIDs;
+              trIDs = "";
+            }
           }
         }
         if (!trIDs.empty()) {
@@ -185,7 +189,7 @@ DataProcessorSpec getPrimaryVertexReaderSpec(bool useMC)
       {"primary-vertex-infile", VariantType::String, "o2_primary_vertex.root", {"Name of the input primary vertex file"}},
       {"vertex-track-matches-infile", VariantType::String, "o2_pvertex_track_matches.root", {"Name of the input file with primary vertex - tracks matches"}},
       {"input-dir", VariantType::String, "none", {"Input directory"}},
-      {"vertex-verbosity", VariantType::Int, 0, {"Print vertex/tracks info: 1) number of contributor and attached, 2) full dump"}}}};
+      {"vertex-verbosity", VariantType::Int, 0, {"Print vertex/tracks info: 1) number of contributor and attached, 2) dump contributors 3) full dump"}}}};
 }
 
 } // namespace vertexing

Original file line number	Diff line number	Diff line change
`@@ -116,17 +116,21 @@ void PrimaryVertexReader::run(ProcessingContext& pc)`
`116`	`116`	`nambig++;`
`117`	`117`	`}`
`118`	`118`	`}`
`119`		`- LOGP(info, "{} : total attached: {}, contributors: {}, ambiguous: {}", GIndex::getSourceName(is), mPV2MatchIdxRef[cnt].getEntriesOfSource(is), ncontrib, nambig);`
	`119`	`+ if (mPV2MatchIdxRef[cnt].getEntriesOfSource(is)) {`
	`120`	`+ LOGP(info, "{} : total attached: {}, contributors: {}, ambiguous: {}", GIndex::getSourceName(is), mPV2MatchIdxRef[cnt].getEntriesOfSource(is), ncontrib, nambig);`
	`121`	`+ }`
`120`	`122`	`if (mVerbose < 2) {`
`121`	`123`	`continue;`
`122`	`124`	`}`
`123`	`125`	`std::string trIDs;`
`124`	`126`	`int cntT = 0;`
`125`	`127`	`for (int i = idMin; i < idMax; i++) {`
`126`		`- trIDs += mPV2MatchIdx[i].asString() + " ";`
`127`		`- if (!((++cntT) % 15)) {`
`128`		`- LOG(info) << trIDs;`
`129`		`- trIDs = "";`
	`128`	`+ if (mVerbose > 2 \|\| mPV2MatchIdx[i].isPVContributor()) {`
	`129`	`+ trIDs += mPV2MatchIdx[i].asString() + " ";`
	`130`	`+ if (!((++cntT) % 15)) {`
	`131`	`+ LOG(info) << trIDs;`
	`132`	`+ trIDs = "";`
	`133`	`+ }`
`130`	`134`	`}`
`131`	`135`	`}`
`132`	`136`	`if (!trIDs.empty()) {`
`@@ -185,7 +189,7 @@ DataProcessorSpec getPrimaryVertexReaderSpec(bool useMC)`
`185`	`189`	`{"primary-vertex-infile", VariantType::String, "o2_primary_vertex.root", {"Name of the input primary vertex file"}},`
`186`	`190`	`{"vertex-track-matches-infile", VariantType::String, "o2_pvertex_track_matches.root", {"Name of the input file with primary vertex - tracks matches"}},`
`187`	`191`	`{"input-dir", VariantType::String, "none", {"Input directory"}},`
`188`		`- {"vertex-verbosity", VariantType::Int, 0, {"Print vertex/tracks info: 1) number of contributor and attached, 2) full dump"}}}};`
	`192`	`+ {"vertex-verbosity", VariantType::Int, 0, {"Print vertex/tracks info: 1) number of contributor and attached, 2) dump contributors 3) full dump"}}}};`
`189`	`193`	`}`
`190`	`194`
`191`	`195`	`} // namespace vertexing`