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

Update jet AOD/RECO event content #7329

Merged
merged 6 commits into from
Jan 26, 2015

Conversation

ahinzmann
Copy link
Contributor

Reduce the number of jet collections in AOD and RECO
Details in the following presentations:
CMS week plenary "JME recent developments":
https://indico.cern.ch/event/354039/session/5/contribution/128/1/material/slides/0.pdf
Presented in the RECO meeting:
https://indico.cern.ch/event/364004/?filterActive=1&showDate=all&showSession=5

In addition two remaining ak5->ak4 transitions are performed in HLTriggerOffline/SUSYBSM and topDQM.

Full list of changes:

Drop ak5 collections (PF, Calo, and Gen)
Increase ak4PFJets and ak4PFJetsCHS pT thresholds from 3 to 5 GeV
Increase ak4CaloJets pT threshold from 3 to 10 GeV
Drop ak8PFJets (only keep ak8PFJetsCHS)
Increase ak8PFJetsCHS pT threshold from 3 to 50 GeV
Drop kT and iterative cone Jets and GenJets. This implies no kt rho calculation (standard = fixed grid rho)
Add ak5CastorJets
Replace pruning with soft drop

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @ahinzmann for CMSSW_7_4_X.

Update jet AOD/RECO event content

It involves the following packages:

DQM/Physics
PhysicsTools/JetCharge
PhysicsTools/PatAlgos
RecoJets/Configuration
RecoJets/JetProducers
RecoLocalCalo/Castor

@nclopezo, @cvuosalo, @monttj, @cmsbuild, @deguio, @slava77, @vadler, @danduggan can you please review it and eventually sign? Thanks.
@rappoccio, @imarches, @yslai, @acaudron, @mmarionncern, @nhanvtran, @schoef, @ferencek, @mariadalfonso, @pvmulder, @TaiSakuma this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.

@slava77
Copy link
Contributor

slava77 commented Jan 22, 2015

@cmsbuild please test

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@slava77
Copy link
Contributor

slava77 commented Jan 26, 2015

+1

for #7329 8b0c388
checked with short matrix that the SUSYDQM HLT_HT* plots are back;
no other changes since the last commit test, as expected.

@davidlange6
Copy link
Contributor

Overriding AT to have this in pre6 - complain if there are issues that we've not identified in the review so far.

davidlange6 added a commit that referenced this pull request Jan 26, 2015
@davidlange6 davidlange6 merged commit 030a21d into cms-sw:CMSSW_7_4_X Jan 26, 2015
@ahinzmann ahinzmann mentioned this pull request Feb 4, 2015
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.

None yet

5 participants