Skip to content

Commit dd98953

Browse files
authored
Merge pull request #1240 from scikit-hep/fix_1238
fix: allow JetResolution to handle additional filename metadata
2 parents 19dd58b + 1ec336d commit dd98953

File tree

4 files changed

+10
-4
lines changed

4 files changed

+10
-4
lines changed

Diff for: src/coffea/jetmet_tools/JetResolution.py

+5-4
Original file line numberDiff line numberDiff line change
@@ -72,15 +72,16 @@ def __init__(self, **kwargs):
7272
)
7373
)
7474
info = name.split("_")
75-
if len(info) != 5:
75+
if len(info) > 6 or len(info) < 5:
7676
raise Exception("Corrector name is not properly formatted!")
77+
offset = len(info) - 5
7778

7879
campaign = _checkConsistency(campaign, info[0])
7980
dataera = _checkConsistency(dataera, info[1])
80-
datatype = _checkConsistency(datatype, info[2])
81-
levels.append(info[3])
81+
datatype = _checkConsistency(datatype, info[2 + offset])
82+
levels.append(info[3 + offset])
8283
funcs.append(func)
83-
jettype = _checkConsistency(jettype, info[4])
84+
jettype = _checkConsistency(jettype, info[4 + offset])
8485

8586
if campaign is None:
8687
raise Exception("Unable to determine production campaign of JECs!")

Diff for: src/coffea/lookup_tools/extractor.py

+1
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
"jec": convert_jec_txt_file,
2323
"jersf": convert_jersf_txt_file,
2424
"jr": convert_jr_txt_file,
25+
"jer": convert_jr_txt_file,
2526
"junc": convert_junc_txt_file,
2627
"ea": convert_effective_area_file,
2728
"pileup": convert_pileup_json_file,
Binary file not shown.

Diff for: tests/test_jetmet_tools.py

+4
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ def jetmet_evaluator():
2727
"* * tests/samples/RegroupedV2_Fall17_17Nov2017_V32_MC_UncertaintySources_AK4PFchs.junc.txt.gz",
2828
"* * tests/samples/Regrouped_Fall17_17Nov2017_V32_MC_UncertaintySources_AK4PFchs.junc.txt",
2929
"* * tests/samples/Spring16_25nsV10_MC_PtResolution_AK4PFPuppi.jr.txt.gz",
30+
"* * tests/samples/Summer23Prompt23_RunCv1234_JRV1_MC_PtResolution_AK4PFPuppi.jr.txt.gz",
3031
"* * tests/samples/Spring16_25nsV10_MC_SF_AK4PFPuppi.jersf.txt.gz",
3132
"* * tests/samples/Autumn18_V7_MC_SF_AK4PFchs.jersf.txt.gz",
3233
]
@@ -312,6 +313,9 @@ def test_jet_resolution(optimization_enabled):
312313
jer_names = ["Spring16_25nsV10_MC_PtResolution_AK4PFPuppi"]
313314
reso = JetResolution(**{name: evaluator[name] for name in jer_names})
314315

316+
check2023_names = ["Summer23Prompt23_RunCv1234_JRV1_MC_PtResolution_AK4PFPuppi"]
317+
_ = JetResolution(**{name: evaluator[name] for name in check2023_names})
318+
315319
print(reso)
316320

317321
resos = reso.getResolution(JetEta=test_eta, Rho=test_Rho, JetPt=test_pt)

0 commit comments

Comments
 (0)