Skip to content
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.

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

Heppy - Replaced PAT photon isolation with RECO isolation #234

Merged
merged 2 commits into from
Feb 11, 2015

Conversation

mmasciov
Copy link
Contributor

@mmasciov mmasciov commented Feb 6, 2015

No description provided.

@mariadalfonso
Copy link
Contributor

@aelwood This PR change the content to the photon isolation (isolation taken from RECO and not PAT now). Please double check.

@aelwood
Copy link
Contributor

aelwood commented Feb 11, 2015

@mariadalfonso the code looks fine to me. As far as I can see it just adds an extra recoIsolation attribute rather than changing it?

@pandolf
Copy link
Contributor

pandolf commented Feb 11, 2015

@aelwood be careful: the branches are overwritten here:

https://github.com/CERN-PH-CMG/cmg-cmssw/pull/234/files#diff-0

@aelwood
Copy link
Contributor

aelwood commented Feb 11, 2015

@pandolf Ah right ok I see. All looks good to me then.

@gpetruc gpetruc merged commit f869cd7 into CERN-PH-CMG:heppy_7_2_2_patch2 Feb 11, 2015
gpetruc added a commit that referenced this pull request Feb 11, 2015
https://github.com/mmasciov/cmg-cmssw into mmasciov-myHeppy_PhotonIso
Heppy - Replaced PAT photon isolation with RECO isolation

Conflicts:
	PhysicsTools/Heppy/python/analyzers/objects/autophobj.py
        (trivial)
gpetruc added a commit that referenced this pull request Feb 11, 2015
gpetruc added a commit that referenced this pull request Feb 11, 2015
lguzzi pushed a commit to lguzzi/cmg-cmssw that referenced this pull request Feb 18, 2019
setesami pushed a commit to setesami/cmg-cmssw that referenced this pull request Nov 6, 2020
…llEras

Switch on DeepFlavour on all 2016 and 2017 samples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants