From 1dad0b6c9e4f8651ee1c96a4e89b0267dd6f4e5d Mon Sep 17 00:00:00 2001
From: Nicole Skidmore <nicola.skidmore@cern.ch>
Date: Tue, 1 Apr 2025 08:38:16 +0000
Subject: [PATCH 1/2] Update file lines.py
---
Hlt/Moore/python/Moore/lines.py | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Hlt/Moore/python/Moore/lines.py b/Hlt/Moore/python/Moore/lines.py
index 05e5440fcc2..5145e77fe43 100644
--- a/Hlt/Moore/python/Moore/lines.py
+++ b/Hlt/Moore/python/Moore/lines.py
@@ -524,7 +524,9 @@ class Hlt2Line(DecisionLine):
"ParticlesSelection" if output_type == "Particles" else output_type
)
- if prefix == "" and output in prdict.values(): # for selective reco case
+ if prefix == "" and str(output) in map(
+ str, prdict.values()
+ ): # for selective reco case
entry_new = [
prefix,
[k for k, v in prdict.items() if output == v][0],
@@ -599,6 +601,7 @@ class Hlt2Line(DecisionLine):
if prefix:
additional_outputs.append((output, (decision_node.name, prefix)))
+ ## Why is the below not an issue?
elif output in prdict.values():
additional_outputs.append((output, None))
else:
--
GitLab
From c76d74ffce55ba867ea356e7424ac668b6651ed7 Mon Sep 17 00:00:00 2001
From: Nicole Skidmore <nicola.skidmore@cern.ch>
Date: Tue, 1 Apr 2025 08:58:16 +0000
Subject: [PATCH 2/2] Apply 1 suggestion(s) to 1 file(s)
Co-authored-by: Carlos Eduardo Cocha Toapaxi <carlos.eduardo.cocha.toapaxi@cern.ch>
---
Hlt/Moore/python/Moore/lines.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Hlt/Moore/python/Moore/lines.py b/Hlt/Moore/python/Moore/lines.py
index 5145e77fe43..3efc2d3368c 100644
--- a/Hlt/Moore/python/Moore/lines.py
+++ b/Hlt/Moore/python/Moore/lines.py
@@ -529,7 +529,7 @@ class Hlt2Line(DecisionLine):
): # for selective reco case
entry_new = [
prefix,
- [k for k, v in prdict.items() if output == v][0],
+ [k for k, v in prdict.items() if str(output) == str(v)][0],
output_type,
"reco",
]
--
GitLab