Skip to content

Backport (of #33966) to 106X: Adding necessary electron structures for low pT part of slimmedElectron collection #33977

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
cmsbuild merged 2 commits into from
Jun 8, 2021
Merged

Backport (of #33966) to 106X: Adding necessary electron structures for low pT part of slimmedElectron collection #33977

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8bde90d
enable necessary low pT structures for b-parking
jainshilpi Jun 4, 2021
311359e
simplified modification
jainshilpi Jun 7, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions PhysicsTools/PatAlgos/python/slimming/slimmedElectrons_cfi.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,6 @@

from Configuration.Eras.Modifier_bParking_cff import bParking
bParking.toModify(slimmedElectrons,dropCorrections="0")
bParking.toModify(slimmedElectrons,dropIsolations="0")
bParking.toModify(slimmedElectrons,dropShapes="0")
bParking.toModify(slimmedElectrons,dropClassifications="0")
Copy link
Contributor

@slava77 slava77 Jun 4, 2021

Choose a reason for hiding this comment

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

Suggested change
bParking.toModify(slimmedElectrons,dropCorrections="0")
bParking.toModify(slimmedElectrons,dropIsolations="0")
bParking.toModify(slimmedElectrons,dropShapes="0")
bParking.toModify(slimmedElectrons,dropClassifications="0")
bParking.toModify(slimmedElectrons,dropCorrections="0", dropIsolations="0", dropShapes="0", dropClassifications="0")

this is simpler to maintain if we need to extend to another modifier

Copy link
Contributor

@slava77 slava77 Jun 7, 2021

Choose a reason for hiding this comment

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

@jainshilpi
are you available to update this PR?

Copy link
Contributor Author

@jainshilpi jainshilpi Jun 7, 2021

Choose a reason for hiding this comment

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

Hi Slava, I missed that comment. Just pushed the modification.

Copy link
Contributor

@slava77 slava77 Jun 7, 2021

Choose a reason for hiding this comment

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

Hi Slava, I missed that comment. Just pushed the modification.

Thank you.