-
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
HLT menu development for 12_4_X
(5/N): HLT V1.3 [12_5_X
]
#38816
HLT menu development for 12_4_X
(5/N): HLT V1.3 [12_5_X
]
#38816
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38816/31191
|
A new Pull Request was created by @missirol (Marino Missiroli) for master. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
enable gpu |
please test |
-1 Failed Tests: RelVals RelVals-INPUT RelVals
RelVals-INPUT
GPU Comparison SummarySummary:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38816/31195
|
Pull request #38816 was updated. @jpata, @missirol, @cmsbuild, @clacaputo, @Martin-Grunewald can you please check and sign again. |
please test The new HLT menu required an update of the function @cms-sw/reconstruction-l2 , this PR and its backport (#38817) now require your signature. |
12_4_X
(5/N): HLT V1.312_4_X
(5/N): HLT V1.3 [12_5_X
]
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ee54f0/26384/summary.html GPU Comparison SummarySummary:
Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+hlt
|
@cms-sw/reconstruction-l2 , please review the minor change in HLT would like to have this PR tested in the next IB if possible. |
I learned that there is some history behind this MkFit customisation, see here and here, incl. an agreement that this customisation should not slow down the integration of HLT updates. @clacaputo @jpata , please consider reviewing and signing. @cms-sw/orp-l2 , I ask to consider this PR for the next IB regardless of the RECO signature (based on the aforementioned agreement). I would prefer to have this PR merged by 11:00, because another HLT PR needs to be prepared on top of this one (but not backported, unlike this one). |
+reconstruction |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
@cms-sw/orp-l2 , a kind ping for the review of this PR. HLT would like to test this in the next IB (23:00). |
+1 |
PR description:
Development of HLT menus for
CMSSW_12_4_X
.This version of the
GRun
HLT menu corresponds to HLT V1.3 for 2022 pp data-taking.This is the PR for the
master
branch, which will then be backported to12_4_X
(the latter is the release cycle currently used for HLT-menu development for Run 3).Includes the integration of the following JIRA tickets.
ParkingDoubleMuonLowMass
(8 PDs, 4 Streams)ParkingDoubleElectronLowMass
(6 PDs, 3 Streams)ParkingSingleMuon
PD for 2022ThroughputService.dqmPathByProcesses
toTrue
SingleMuon+DoubleMuon=Muon
,JetHT+MET=JetMET
)HLT_Photon30EB_TightID_TightIso
from 150 to 6drop *
statement to AlCaPPS EventContentmaxEta
cut inAlCa_RPCMuonNormalisation
triggersPFJet110
control triggerHLTMonitor
PD and stream for HLT-PNET monitoringIn addition, the input file used for the HLT-Validation tests in IBs is updated to a recent collisions run:
/store/data/Run2022B/HLTPhysics0/RAW/v1/000/355/558/00000/cd851cf4-0fca-4d76-b80e-1d33e1371929.root
PR validation:
TSG tests.
If this PR is a backport, please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
12_4_X