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

CMG updates to Heppy #17

Merged
merged 50 commits into from
Feb 17, 2015
Merged

Conversation

gpetruc
Copy link

@gpetruc gpetruc commented Feb 10, 2015

Updates to Heppy that have already been integrated from CMG side

@arizzi @mariadalfonso

gpetruc and others added 30 commits January 16, 2015 17:21
Pull cms-sw#198 into Heppy branch (as it was supposed to be)
Heppy - Fixed metNoMu to still subtract the muons even when not recalibr...
Heppy - fixed QGLikelihood for 72X
…el, and add -o and getHeppyOption to allow passing command line options to cfgs for testing
Upstream Heppy updates: logger verbosity and minor fixes
Heppy - implemented full5x5 photon clustershape methods
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)
Conflicts:
	PhysicsTools/HeppyCore/scripts/heppy_batch.py
…ibrating

Conflicts:
	PhysicsTools/Heppy/python/analyzers/objects/METAnalyzer.py
…ibrating

Conflicts:
	PhysicsTools/Heppy/python/analyzers/objects/METAnalyzer.py
…ibrating

Conflicts:
	PhysicsTools/Heppy/python/analyzers/objects/METAnalyzer.py
…n to Imperial College

Conflicts:
	PhysicsTools/Heppy/python/analyzers/objects/METAnalyzer.py
	PhysicsTools/HeppyCore/scripts/heppy_batch.py
gpetruc and others added 20 commits February 11, 2015 18:27
Cherry picked Heppy only commits related to PR 242
Heppy: Update tau object/analyzer/filler to latest POG recommendations
…PR18

Heppy CMSSW Preprocessor, and performance improvement in LHEAnalyzer
* allow specifying an arbitrary isolation cut function (isoCut), to
  cover choices of radii, relative vs absolute iso, etc.
* allow computation of miniIsolation, as presented in
    https://indico.cern.ch/event/368826/
* rearrange code to first compute all inclusive leptons,
  then compute loose ones. this allows e.g. vetoing
  inclusive leptons in isolation cones of other leptons
…o keep the lepton or the jet during cleaning, save discarded objects, and make more consistent cleaning of genJets
…s_PRs_223_227_noEleMVA

Port some PRs into Heppy branch
Heppy: jets and photons cleaning and met type1 with chs jets
Heppy optimizations and an improved time report
@gpetruc
Copy link
Author

gpetruc commented Feb 16, 2015

Hi,

Any chance of getting this merged soon?

There's more to come from our side, in addition to #19, as you can see from the cmgtools github
( https://github.com/CERN-PH-CMG/cmg-cmssw/pulls?q=is%3Apr+label%3A%22to+propagate+to+Colin%22+ ).
Should I add them here, rather than waiting for this to be done to make a new one?

Giovanni

@cbernet
Copy link
Owner

cbernet commented Feb 16, 2015

Hi,

You can add to this PR.

C

Le 16 févr. 2015 à 14:29, Giovanni Petrucciani [email protected] a écrit :

Hi,

Any chance of getting this merged soon?

There's more to come from our side, in addition to #19, as you can see from the cmgtools github
( https://github.com/CERN-PH-CMG/cmg-cmssw/pulls?q=is%3Apr+label%3A%22to+propagate+to+Colin%22+ ).
Should I add them here, rather than waiting for this to be done to make a new one?

Giovanni


Reply to this email directly or view it on GitHub.

@gpetruc
Copy link
Author

gpetruc commented Feb 16, 2015

Ok, I have added everything that exists now in the heppy branch in the CMGTools repository, and updated the description of the pull request to reflect this.
Note that the e/gamma MVA electron id is still left outside in #19

cbernet added a commit that referenced this pull request Feb 17, 2015
Hi Giovanni, 
Thank you very much. I'll merge faster next time.
@cbernet cbernet merged commit 4838fdf into cbernet:heppy_7_2_2_patch2 Feb 17, 2015
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.

8 participants