-
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
L1T uGT: Three body invariant mass for muons #33911
L1T uGT: Three body invariant mass for muons #33911
Conversation
…0_X starting from CMSSW_12_0_X_2021-05-24-2300
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33911/22979 |
A new Pull Request was created by @elfontan (Elisa Fontanesi) for master. It involves the following packages: L1Trigger/L1TGlobal @cmsbuild, @rekovic, @cecilecaillol can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: Build ClangBuild BuildI found compilation warning when building: See details on the summary page. Clang BuildI found compilation warning while trying to compile with clang. Command used:
See details on the summary page. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33911/23004 |
Pull request #33911 was updated. @cmsbuild, @rekovic, @cecilecaillol can you please check and sign again. |
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33911/23113 |
Pull request #33911 was updated. @cmsbuild, @rekovic, @cecilecaillol can you please check and sign again. |
thanks @elfontan |
+l1 |
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. @silviodonato, @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-92449d/15715/summary.html Comparison SummarySummary:
|
+1 |
PR description:
Development of a new condition class to deal with a three-body correlation condition (starting from CorrCondition and CorrWithOverlapRemovalCondition as a model): CorrThreeBodyCondition and CorrelationThreeBodyTemplate.
Based on the version of the UTM libraries utm_0.8.1: https://gitlab.cern.ch/cms-l1t-utm/utm/-/tree/utm_0.8.1.
New parseCorrelation function needed in the TriggerMenuParser. Minor changes needed in the L1TGlobalProducer, GlobalBoard, TruggerMenu, GlobalCondition, GlobalDefinitions.
PR validation:
CMSSW release used for tests: CMSSW_12_0_X_2021-05-24-2300.
STEPS:
- Private production of a small RAW-DIGI ZZTo4Mu sample (100 events) using Run 3 conditions
- Production of a dedicated HLT menu in ConfDB including FirstPath, FinalPath, and a generic TripleMu seed (HLT_TripleMu_10_5_5_DZ_v10) with modified L1 seeds as Logical Expression:
/users/elfontan/ThreeMuInvMassTest/TriMuInvarianMass1120/V4
- HLT configuration and uGT emulator run with a dedicated L1 test menu:
[... --l1-emulator uGT --mc --l1Xml L1Menu_test_mass_3_body_reduced_v2.xml"]
- scram build code-format #Ensure that the code conforms to CMSSW standards
- scram b -j8
- runTheMatrix.py -l 11634.0