-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Skim for Delayed Jets, MET cut instead of HT #37808
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37808/29726
|
A new Pull Request was created by @rtschmitz for master. It involves the following packages:
@cmsbuild, @bbilin, @kskovpen, @jordan-martins can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
One additional comment I wanted to make: This PR is complementary to PR#37749, which provides a general MET>200 skim. This skim lowers the MET threshold for delayed jets to MET>120. |
+pdmv |
This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a8c7b6/25411/summary.html Comparison SummarySummary:
|
+1 |
paths = (EXODelayedJetMETPath), | ||
content = skimRawAODContent.outputCommands, | ||
selectEvents = cms.untracked.PSet(), | ||
dataTier = cms.untracked.string('AOD') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this really supposed to have been an AOD
? @sam7k9621 @davidlange6 @rappoccio
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the analysis needs access to the ecal rechits that are only available in AOD
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other skims use USER
, right? @sam7k9621 @davidlange6 @rappoccio please comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the late reply.
I think it can remain ‘AOD’ since there is no customized event content
PR description:
This PR adds one RECO level skim that saves all standard AOD collections. This PR is similar to PR#37509, which added a skim over delayed jets using a set of HLT paths, except this skim uses a cut on MET rather than on HT. It does this using the producer/filter for delayed jets in CMSSW and skimming over the HLT path "HLT_PFMET120_PFMHT120_IDTight_v*".
Tested with high-Pt QCD samples /store/relval/CMSSW_12_3_0_pre6/RelValQCD_Pt_1800_2400_14/GEN-SIM-RECO/123X_mcRun3_2021_realistic_v11-v2/, the skimmed AOD file size was 25MB (68 events) and the unskimmed AOD file size was 1.9GB (9000 events).
PR validation:
Ran with CMSSW_12_4_0_pre1 and used the config file "test_DelayedJetMET_cfg.py" which is included in this PR.