Skip to content

Propagating changes made by Davide and others long ago to run on Run 2#43

Open
cippy wants to merge 11 commits intoWMass:WmassNanoProd_10_6_26from
erc-asymow:WmassNanoProd_10_6_26_fullRun2
Open

Propagating changes made by Davide and others long ago to run on Run 2#43
cippy wants to merge 11 commits intoWMass:WmassNanoProd_10_6_26from
erc-asymow:WmassNanoProd_10_6_26_fullRun2

Conversation

@cippy
Copy link
Copy Markdown

@cippy cippy commented Mar 26, 2026

PR description:

As agreed with David, I am just propagating the commits from erc-asimow/cmssw to synchronise the main repository. This should be needed to run on Run 2 samples

PR validation:

if this PR is a backport please specify the original PR:

Before submitting your pull requests, make sure you followed this checklist:

#the94X miniAOD V2 had a bug in the scale and smearing for electrons in the E/p comb
#therefore we redo it but but we need use a new name for the userFloat as we cant override existing userfloats
#for technical reasons
for modifier in run2_nanoAOD_94XMiniAODv1,run2_nanoAOD_94XMiniAODv2,run2_miniAOD_80XLegacy,run2_nanoAOD_102Xv1,run2_nanoAOD_106Xv1,run2_nanoAOD_106Xv2,run2_nanoAOD_LowPU:
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mmh, I am not sure it is correct to remove the lowPU piece from here

@cippy
Copy link
Copy Markdown
Author

cippy commented Mar 26, 2026

I don't know whether all the changes from this PR are actually meaningful, but those who dealt with this part of the code have left the group :-/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants