Add LUT for Kalman algo of KMTF - #2
Conversation
|
A new Pull Request was created by @zhenbinwu for branch main. @BenjaminRS, @akritkbehera, @cmsbuild, @iarspider, @quinnanm, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-64cc2f/55087/summary.html Comparison SummarySummary:
|
|
+l1 |
|
please test with cms-sw/cmssw#51111 This PR contains a new data files needed by cms-sw/cmssw#51111 |
|
-1 Failed Tests: Build The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Failed BuildI found compilation error when building: @@@@ ----> OK EDM Class Transients DataFormatsL1Scouting >> Checking EDM Class Version for src/DataFormats/MuonDetId/src/classes_def.xml in DataFormatsMuonDetId @@@@ ----> OK EDM Class Transients DataFormatsL1TCalorimeter >> Checking EDM Class Version for src/DataFormats/MuonReco/src/classes_def.xml in DataFormatsMuonReco @@@@ ----> OK EDM Class Transients DataFormatsL1TCalorimeterPhase2 error: class 'l1t::KMTFTrack' has a different checksum for ClassVersion 4. Increment ClassVersion to 5 and assign it to checksum 2306202566 >> Checking EDM Class Version for src/DataFormats/MuonSeed/src/classes_def.xml in DataFormatsMuonSeed Suggestion: You can run 'scram build updateclassversion' to generate src/DataFormats/L1TMuonPhase2/src/classes_def.xml.generated with updated ClassVersion gmake: *** [tmp/el9_amd64_gcc13/edm_checks/libDataFormatsL1TMuonPhase2.so] Error 1 >> Checking EDM Class Version for src/DataFormats/NanoAOD/src/classes_def.xml in DataFormatsNanoAOD @@@@ ----> OK EDM Class Transients DataFormatsL1TGlobal |
|
Increment ClassVersion of KMTFTrack in cms-sw/cmssw#51111 |
|
please test with cms-sw/cmssw#51111 |
|
-1 Failed Tests: RelVals RelVals-INPUT Failed RelVals
Expand to see more relval errors ...Failed RelVals-INPUT
Expand to see more relval errors ...
|
I am confused. packedGainLUTs.root is already in this repo. |
|
I am looking in to it |
|
please test with cms-sw/cmssw#51111 |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-64cc2f/55175/summary.html Comparison SummarySummary:
|
|
REMINDER @ftenchini, @sextonkennedy, @mandrenguyen: This PR was tested with cms-sw/cmssw#51111, please check if they should be merged together |
|
+1 |
|
merge |
Add a data file from cms-sw/cmssw#51111
Will remove the data file from CMSSW once this is merged. Thanks