-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Updating TopMuEG skim with latest trigger paths for 2023 collisions #42057
Conversation
A new Pull Request was created by @soureek (Soureek Mitra) for CMSSW_13_0_X. It involves the following packages:
@cmsbuild, @bbilin, @sunilUIET, @kskovpen can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
Hi @soureek , thanks! Could you also update this PR to 13_1_X and 13_2_X for synchronization? |
@youyingli Created new PRs for 13_1_X and 13_2_X updates. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-802150/33360/summary.html Comparison SummarySummary:
|
+pdmv |
backport of #42066 |
+pdmv |
This pull request is fully signed and it will be integrated in one of the next CMSSW_13_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_13_2_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
@@ -550,6 +539,7 @@ | |||
'keep TotemVFATStatusedmDetSetVector_ctppsDiamondRawToDigi_*_*', | |||
'keep CTPPSDiamondRecHitedmDetSetVector_ctppsDiamondRecHits_*_*', | |||
'keep CTPPSDiamondLocalTrackedmDetSetVector_ctppsDiamondLocalTracks_*_*', | |||
'keep TotemTimingLocalTrackedmDetSetVector_diamondSampicLocalTracks_*_*', |
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.
The following keep statements were present in the master version of this PR, but missing here:
'keep TotemFEDInfos_totemT2Digis_*_*',
'keep TotemT2DigiedmNewDetSetVector_totemT2Digis_*_*',
'keep TotemVFATStatusedmDetSetVector_totemT2Digis_*_*',
Since this is a test script it does not prevent getting merged, but probably it is something you want to have a look at, @soureek
+1 |
PR description:
Updating TopMuEG skim for 2023 collisions with the latest trigger paths. The files DPGAnalysis/Skims/python/TopMuEGSkim_cff.py is updated with the latest paths.
PR validation:
PR is intended for CMSSW_13_0_X and the changes have tested with "/RelValTTbar_14TeV/CMSSW_13_0_0-130X_mcRun3_2022_realistic_v2-v1/GEN-SIM-RECO" dataset. No issue was observed.