-
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
131X update for TopMuEG skim #42064
131X update for TopMuEG skim #42064
Conversation
A new Pull Request was created by @soureek (Soureek Mitra) for CMSSW_13_1_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
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-358cad/33361/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_1_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 +542,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
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.
+1 |
PR description:
Updating HLT paths for TopMuEG skim in 13_1_X.
Related to #42057