Skip to content

Fix bug in pileup re-weighting auto-config code

Chris Meyer requested to merge cjmeyer/athena:prw-autoconfig-fix into master

I was excited to see the recent Campaign MR !56759 (merged) from @tadej land in 22.2.91! I gave it try with the pileup re-weighting auto config and noticed the resulting path was:

dev/PileupReweighting/share/DSID601xxx/pileup_Campaign.MC21a_dsid601189_FS.root

which can't be found. This is because campaign is now an Enum instead of a string. After building the pileup config filename with f'{campaign.value}' I get:

dev/PileupReweighting/share/DSID601xxx/pileup_mc21a_dsid601189_FS.root

and everything works.

In case this is already known / fixed elsewhere / intended behavior, feel free to close this MR...

Merge request reports

Loading